Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modals not rendering maximize/minimize glyphs #413

Closed
ahacking opened this issue Mar 28, 2023 · 4 comments
Closed

Modals not rendering maximize/minimize glyphs #413

ahacking opened this issue Mar 28, 2023 · 4 comments

Comments

@ahacking
Copy link

Environment

Make sure you are using the latest LiveView and Dashboard versions before continuing.

  • Elixir version (elixir -v): 1.14.2
  • Phoenix version (mix deps): 1.7.2
  • Phoenix LiveView version (mix deps): 0.18.18
  • Phoenix Dashboard version (mix deps): 0.7.2
  • Operating system: Linux
  • Browsers you attempted to reproduce this bug on (the more the merrier): Firefox 1110.0.1, Android Chrome

Actual behavior

The Maximize and minimize glyph is not rendered in modal title.

It appears to be an issue with the inline font not providing the required code points U+1F5D6 and U+1F5D7 used here:


and here:

Expected behavior

Maximize/Minimize glyph is rendered by the browser.

@josevalim
Copy link
Member

@fmterrorf can you plaese give us a hand here? The icons may not have been committed as part of #367? Do you remember?

@josevalim
Copy link
Member

Ah, it seems we are missing the font with those Unicode characters. So we need to address it or revert it. :(

@fmterrorf
Copy link
Contributor

fmterrorf commented Mar 31, 2023

@fmterrorf can you plaese give us a hand here? The icons may not have been committed as part of #367? Do you remember?

I got it from here https://stackoverflow.com/a/60570109. I can have a look on this. Maybe we can just use an SVG like what's done here

<svg class="hint-icon" viewBox="0 0 44 44" fill="none" xmlns="http://www.w3.org/2000/svg">

@josevalim
Copy link
Member

@fmterrorf ah, the SO link is great. I will just use minus and square for now. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants