The toggle button is only displayed when auto-hide is not set (should it be displayed at all times?).

Is autohide even needed if this feature is present? We could conceivably just take it out.

Is autohide even needed if this feature is present? We could conceivably just take it out.

Good idea. I personally wouldn't mind, but I think it is better if we have some input from users other than ourselves, so I created a poll here : https://www.reddit.com/r/airsonic/comments/f5yeis/poll_how_do_you_use_the_airsonic_play_queue.

I like this change. Could animate it too, but I'm in favor of it.

Don't have an opinion about the iconography (others more visually inclined may have better opinions), but overall this looks pretty good to me.

I still like having the option of auto-hide, but the toggle is a good improvement.

A few things since this was merged on airsonic-advanced a while back.

• The height of the toggle hide is sometimes wonky. You may need to re-account for some of the pixels with the framesets heights, especially when they redraw
• The frameset height setter option itself has a bug where you're not rounding the height to the nearest pixel. At least on Chrome, setting the rows to something like "*, 270.33333333" results in a weird and seemingly arbitrary height. This bug is present here on the airsonic fork: https://github.com/airsonic/airsonic/blob/master/airsonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp#L148
  It was fixed in this commit: airsonic-advanced/airsonic-advanced@3d5e8c6 as part of a faster table rendering commit