code.pyret.org icon indicating copy to clipboard operation
code.pyret.org copied to clipboard

Font size picker can wrap

Open jpolitz opened this issue 4 years ago • 3 comments
trafficstars

I saw this in an embedded context (when on an iframe in VMT), but it doesn't look great:

image

jpolitz avatar Sep 30 '21 18:09 jpolitz

Also, let me repeat the issue I raised at https://github.com/brownplt/code.pyret.org/pull/377#issuecomment-847822736, in case they can be fixed together.

The font size controls in the main menu are greyed out, making it look like the controls are disabled. However, they actually are not. The situation is slightly better in master because when you hover over it, the color changes. In horizon, the color doesn't change at all on hovering.

sorawee avatar Sep 30 '21 19:09 sorawee

I intend to open a PR shortly to:

  • Prevent wrapping
  • Make the buttons clearer (no gray!)
  • Prevent weird font sizes with long decimals

asolove avatar Apr 29 '22 14:04 asolove

This is fixed as part of #434 .

asolove avatar May 01 '22 23:05 asolove

Closing, since the PR was merged and the font size picker no longer wraps

schanzer avatar Jan 11 '24 15:01 schanzer