Skip to content

Conversation

@GretaD
Copy link
Contributor

@GretaD GretaD commented Aug 24, 2022

Fixes #7098

for some reasons, the active state on envelope is not handled by vue router, i dont know why.
Adding the :active on listitem on evelope makes it work again(aka using the style from nc/vue component without forcing style on mail side)

Signed-off-by: greta <gretadoci@gmail.com>
@ChristophWurst
Copy link
Member

for some reasons, the active state on envelope is not handled by vue router, i dont know why.

@JuliaKirschenheuter @st3iny can one of you have a look? I strongly believe that the active state should be handled by the Vue router already. Maybe our styling just uses the wrong active state. If anything, this should be fixed in nc/vue.

@ChristophWurst
Copy link
Member

nextcloud-libraries/nextcloud-vue#3147 and #7128 fix the problem at its root.

@ChristophWurst ChristophWurst deleted the fix/active-state-envelope branch September 1, 2022 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Envelope hover radius doesn't match active thread radius

3 participants