-
Notifications
You must be signed in to change notification settings - Fork 36
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
Replace the crown with both Mac and Windows logo? #30
Comments
It's true, we could probably create our own SVG which clearly indicates both? |
Hi, Since there are only two, I think it would look better to have both icons than a single icon combined. |
Yeah @KallunWillock suggested this to me a while back too. As in literally: - [![p_win]](#-) [![p_mac]](#-) [![a_all]](#-) SomeLibrary - Description I'm not a massive fan of this though because I see these as 2 columns, I feel that the proposed chaining will be more difficult to read unless there is a "Not mac" symbol. |
For clarity these are mockups of our options: At this scale it does appear as though the merged mac/windows icon is pretty difficult to see, though maybe good enough to distinguish. Option C is the easiest to read, and to be fair we don't need a specific not symbol per OS, it can just be a generic "disabled" symbol with a different tooltip. |
Option C seems like the easiest to read indeed and I think having seperate "not" symbols is also good for clarity. |
Currently the crown () is the symbol for "All plateforms", however it is not a very intuitive symbol. Perhaps a better option would be to just have both the Mac and Windows logo when an entry works for both?
Or at least, I think we should change the tooltip from "Compatible on all platforms" to "Compatible on both Mac and Windows" since VBA is not compatible with the Unix plateform or any mobile plateforms.
The text was updated successfully, but these errors were encountered: