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

oh-icon component improvements, Iconify icon support #1149

Merged
merged 2 commits into from
Aug 29, 2021
Merged

Conversation

ghys
Copy link
Member

@ghys ghys commented Aug 28, 2021

This allows the oh-icon to display icons from other sources than the
OH icon servlet; bundled Framework7/Material icons can also be displayed
by this component (use f7: or material: prefixes).
Specifying no prefix will assume an OH icon for backward compatibility.

This also introduces Iconify (https://icon-sets.iconify.design) as an
icon source, allowing access to 100,000+ icons from various icon sets,
to display them in widgets like cards or floorplan markers.

The Iconify option requires Internet connectivity on the client to access
the external API but the underlying library will cache the icons in
local/session storage to limit the requests and speed up the rendering.

Update/fix documentation and plan marker default icon registration.

Closes #1081 (as the mdi icon set is available though Iconify).

Signed-off-by: Yannick Schaus github@schaus.net

This allows the oh-icon to display both from more sources than the
OH icon servlet; bundled Framework7/Material icons can also be displayed
by this component (use `f7:` or `material:` prefixes).
Specifying no prefix will assume an OH icon for backward compatibility.

This also introduces Iconify (https://icon-sets.iconify.design) as an
icon source, allowing access to 100,000+ icons from various icon sets,
to display them in widgets like cards or floorplan markers.

The Iconify option requires Internet connectivity on the client to access
the external API but the underlying library will cache the icons in
local/session storage to limit the requests and speed up the rendering.

Update/fix documentation and plan marker default icon registration.

Signed-off-by: Yannick Schaus <github@schaus.net>
@ghys ghys requested a review from a team as a code owner August 28, 2021 18:00
@relativeci
Copy link

relativeci bot commented Aug 28, 2021

Job #190: Bundle Size — 10.49MB (~-0.01%).

d22cdeb vs 3a0b77b

Changed metrics (2/8)
Metric Current Baseline
Initial JS 1.63MB(~-0.01%) 1.63MB
Cache Invalidation 15.73% 25.17%
Changed assets by type (1/7)
            Current     Baseline
JS 8.23MB (~-0.01%) 8.23MB

View Job #190 report on app.relative-ci.com

Signed-off-by: Yannick Schaus <github@schaus.net>
@hmerk
Copy link

hmerk commented Aug 29, 2021

@ghys Great job Yannick !!

@ghys ghys merged commit 01cee7b into openhab:main Aug 29, 2021
@ghys ghys deleted the iconify branch August 29, 2021 22:59
@kaikreuzer kaikreuzer added the bug Something isn't working label Sep 5, 2021
@kaikreuzer kaikreuzer added this to the 3.2 milestone Sep 5, 2021
@ghys ghys added the main ui Main UI label Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working main ui Main UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FR: Add materialdesignicons to MainUI
3 participants