-
Notifications
You must be signed in to change notification settings - Fork 14
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
IBX-6905: Changing Country in Corporate Account form removes current data without any warning #1182
Conversation
@@ -0,0 +1,22 @@ | |||
@mixin spinner($size, $line-size, $color: $ibexa-color-info) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's backported from 4.6 so I left it as it is with hope there will be no conflicts, but I started to wonder if we should treat ibexa-color-info as default for spinner color - as I check it now on 4.6, we have 3 usages of info, 4 (or even 5 with this issue) of primary and one for white.
Do we want to leave it as it is for BC and counter-intuitive, or we'd rather to change it to primary? (which would require more work with upmerge, as these three places where we use info (as default) would need to be adjusted.
98fe4cc
to
40c6937
Compare
40c6937
to
308f09d
Compare
308f09d
to
6acd33a
Compare
…data without any warning
6acd33a
to
24f0f15
Compare
|
QA Approved on Ibexa DXP Commerce 4.5-dev. |
v4.5
Backporting spinner mixin for fieldtype-address fix
Checklist:
@ibexa/engineering
).