-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Rotate refresh icon #332
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
Rotate refresh icon #332
Conversation
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.
Looks nice, thanks 😄
src/devtools/App.vue
Outdated
@@ -78,7 +79,13 @@ export default { | |||
} | |||
}, | |||
refresh () { | |||
this.refreshRotate += 180 | |||
const refleshIcon = this.$el.querySelector('.refresh-icon') |
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.
you can use a ref on the element instead
src/devtools/App.vue
Outdated
bridge.send('refresh') | ||
bridge.on('flush', () => { |
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.
I forgot to say that the bridge.on should be setup only once
Hi @posva , what do you think about changing the icon? An idea would be to leave the same chrome refresh icon. |
src/devtools/global.styl
Outdated
|
||
@-moz-keyframes rotate | ||
0% | ||
-moz-transform rotate(0deg) |
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.
No need for prefixes because stylus will add them. So keeping the @keyframes
rule with only transform
is enough
@posva , what do you think of these latest changes? |
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.
It looks good, thanks 🙂
Maybe it is not the best way to implement, but it would be nice to have a behavior that indicase the refresh.