remove "hoverable" class from divs #12408
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Findus23
added
the
c: Design / UI
|
That was me who added it. I found it quite useful especially when there are many boxes but I have no strong opinion here and it can be removed for sure. I can see that it possibly could be annoying. |
|
I also don't have such a strong opinion on it, so I'd wait for more other opinions. |
|
I also think it's quite useful and a bit of interaction/animation in the product. will close it for now but thanks for the suggestion. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
One thing that is bugging me a bit since Piwik 3 is that the cards in the visitor log are all animated on hover.
In my opinion such a strong animation should be limited to small objects and it always indicates that one can interact with it as it's a link or button.
I am not sure why the visitor log cards "move" as no other similar cards (sidebar/other reports) do so apart from the "manage websites" list for which I'd propose the same change.