| 10342 | icon.toggleClass('openIcon closedIcon'); | 10342 | icon.toggleClass('openIcon closedIcon'); |
| 10343 | drawer.toggleClass('openDrawer closedDrawer'); | 10343 | drawer.toggleClass('openDrawer closedDrawer'); |
| 10344 | | 10344 | |
| 10345 | //commented code is to prevent scrollbars showing during transition, if we want to do that | | |
| 10346 | //'scrollableInner' only exists on samplers panel though, so will need different logic for each drawer | | |
| 10347 | //this does create an undesireable leftward width retraction of about 15px if a natural scrollbar pops after transition | | |
| 10348 | //however, if we don't prevent the transition scrollbar, we get a 15px expansion right when it is no longer needed | | |
| 10349 | //we get to pick our poison. (or...we go with just opacity transitions and stop caring about height and layout shifts.) | | |
| 10350 | //only true fix for this is to make scrollbars always show (overflow-y:scroll), and style them so they are transparent until div hover | | |
| 10351 | //this creates a 15px perma-padding on the right of the div tho, even when not needed | | |
| 10352 | //example of this can be seen on STMP's pastChat list | | |
| 10353 | | | |
| 10354 | //drawer.find('.scrollableInner').addClass('overflowHidden'); //prevents scrollbars | | |
| 10355 | //await delay(250); //this duration is presumed | | |
| 10356 | //drawer.find('.scrollableInner').removeClass('overflowHidden'); //restore after transition | | |
| 10357 | | | |
| 10358 | if (targetDrawerID === 'right-nav-panel') { | 10345 | if (targetDrawerID === 'right-nav-panel') { |
| 10359 | favsToHotswap(); | 10346 | favsToHotswap(); |
| 10360 | $('#rm_print_characters_block').trigger('scroll'); | 10347 | $('#rm_print_characters_block').trigger('scroll'); |