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.

Sign up for GitHub

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

Jump to bottom

The Italic feature icon is serif while other icons are sans-serif #928

Closed
jodator opened this issue Mar 29, 2018 · 4 comments
Closed

The Italic feature icon is serif while other icons are sans-serif #928

jodator opened this issue Mar 29, 2018 · 4 comments
Labels
package:basic-styles package:ui resolution:expired This issue was closed due to lack of feedback. status:discussion type:improvement This issue reports a possible enhancement of an existing feature.

Comments

@jodator
Copy link
Contributor

jodator commented Mar 29, 2018

While reading #910 I've have one more thing to add (sorry for that but @wwalc told me to report this).

So the Italic feature icon is a serif one while other icons are sans-serif

selection_005

I don't see a better solution though.
@jodator jodator added type:improvement This issue reports a possible enhancement of an existing feature. status:discussion package:basic-styles package:ui labels Mar 29, 2018
@scofalik
Copy link
Contributor

Well, "slash" icon might be not the best idea.

@jodator
Copy link
Contributor Author

jodator commented Mar 29, 2018 •
edited
Loading

Well, "slash" icon might be not the best idea.

Yeah I know that's probably a reason but since we're looking at icons... 😉

edit: it always can be a lower cased one: i a bit better then I I think.

@Reinmar
Copy link
Member

Reinmar commented Mar 29, 2018

Italicised lowercased "i" looks like an "info point" ;) We've been there.

@mlewand
Copy link
Contributor

mlewand commented Sep 2, 2019

Since this issue issue is inactive for a quite some time, and we were unable to come up with a constructive proposal I'll close it for now.

@mlewand mlewand closed this as completed Sep 2, 2019
@mlewand mlewand added the resolution:expired This issue was closed due to lack of feedback. label Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
package:basic-styles package:ui resolution:expired This issue was closed due to lack of feedback. status:discussion type:improvement This issue reports a possible enhancement of an existing feature.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Reinmar @jodator @scofalik @mlewand