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

Check config before allowing oh-icon style customization #1243

Merged
merged 3 commits into from
Dec 18, 2021
Merged

Conversation

ghys
Copy link
Member

@ghys ghys commented Dec 18, 2021

#1233 offers style customization properties for f7 and iconify icons but they end
up conflicting with those set by the f7-icon & iconify-icon components themselves.

Fixes #1242.

Signed-off-by: Yannick Schaus [email protected]

#1233 offers style customization properties for f7 and iconify icons but they end
up conflicting with those set by the `f7-icon` & `iconify-icon` components themselves.
@ghys ghys requested a review from a team as a code owner December 18, 2021 17:52
@relativeci
Copy link

relativeci bot commented Dec 18, 2021

Job #298: Bundle Size — 10.66MB (~+0.01%).

560a5a2 vs d3bfeed

Changed metrics (2/8)
Metric Current Baseline
Initial JS 1.66MB(~+0.01%) 1.66MB
Cache Invalidation 15.76% 18.34%
Changed assets by type (1/7)
            Current     Baseline
JS 8.61MB (~+0.01%) 8.61MB

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

@ghys ghys changed the title Revert #1233 - oh-icon style customization Check config before allowing oh-icon style customization Dec 18, 2021
@ghys
Copy link
Member Author

ghys commented Dec 18, 2021

Finally it appears checking that the oh-icon has a config set is enough to solve both #1242 and #1183.

Signed-off-by: Yannick Schaus <[email protected]>
@ghys
Copy link
Member Author

ghys commented Dec 18, 2021

Merging as a critical bugfix, thanks @digitaldan for reporting it.

@ghys ghys merged commit abb43cf into main Dec 18, 2021
@ghys ghys added bug Something isn't working main ui Main UI labels Dec 18, 2021
@ghys ghys added this to the 3.2 milestone Dec 18, 2021
@ghys ghys deleted the revert-1233 branch June 22, 2022 13:58
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.

Missing f7 / material icons in latest builds
1 participant