Skip to content
This repository has been archived by the owner on Apr 17, 2024. It is now read-only.

add :active style of ae-button as :hover effekt for desktop support #216

Closed
wants to merge 1 commit into from

Conversation

emilsomm
Copy link

@sadiqevani
Copy link
Contributor

sadiqevani commented Nov 27, 2018

@emilsomm

I'm not sure if the :hover state should have an opacity, the issue here is that when the button is also clicked and remains :focused it will apply the opacity, which in Mobile versions might keep the state like that until the user clicks somewhere else.

I think for the button state we should talk with @raymundjacobs and have two different states:

  • :focused state should maybe apply opacity or inner shadow
  • :hover state should maybe apply an increased box-shadow

@emilsomm
Copy link
Author

emilsomm commented Nov 27, 2018 via email

@sadiqevani
Copy link
Contributor

I'm closing this PR, we can come back at this issue later.

@sadiqevani sadiqevani closed this Jan 11, 2019
@sadiqevani sadiqevani deleted the feature/add-hover-to-button branch January 11, 2019 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants