-
Task
-
Resolution: Unresolved
-
P3: Somewhat important
-
None
-
Some future release
-
None
The QIcon theme machinery/APIs/docs could use some revamp. The original design was based on the freedesktop icon spec, but we should not limit ourselves to that. For example, macOS 11+ now has a comprehensive icon set known as SF Symbols https://developer.apple.com/sf-symbols/
Some steps/ideas:
- Let the string based QIcon::fromTheme look up the strings directly in the underlying theme. On freedesktop platforms that will be 1:1 with what we have today. On macOS the names from SF Symbols will be mapped.
- This behavior change should be fine, as no non-freesktop platform implements this today anyways
- Add a QIcon::StandardIcon enum and fromTheme overload that matches the existing freedesktop spec, mapping to the same names on freedestkop platforms, and to the relevant names on SF Symbols.
- Guard the QIconCacheGtkReader behind a GTK configure ifdef, or possibly move to the Gtk3 platform theme if possible
- relates to
-
QTBUG-102110 QIcon::fromTheme fallback regression
-
- Closed
-
For Gerrit Dashboard: QTBUG-102346 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
490940,18 | Add initial implementation of macOS and iOS icon theme implementations | dev | qt/qtbase | Status: MERGED | +2 | 0 |
490981,18 | Add initial implementation of a Windows icon engine | dev | qt/qtbase | Status: MERGED | +2 | 0 |
492063,11 | Initial implementation of an iOS icon engine | dev | qt/qtbase | Status: ABANDONED | 0 | 0 |
492064,19 | Add initial implementation of an Android icon engine | dev | qt/qtbase | Status: MERGED | +2 | 0 |
521004,4 | IconImage: make aware of platform icon engines | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
521008,5 | Manual iconbrowser test: Add a Quick test tab | dev | qt/qtbase | Status: MERGED | +2 | 0 |
605264,22 | Refactor shared code of font-based icon engines | dev | qt/qtbase | Status: MERGED | +2 | 0 |
605265,22 | QFontIconEngine: render named glyphs as icons, if possible | dev | qt/qtbase | Status: MERGED | +2 | 0 |
605267,21 | QFontEngine: add API to get the name of a glyph | dev | qt/qtbase | Status: MERGED | +2 | 0 |