Skip to content
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

Jump to bottom

High importance button show no hovered state #1785

Closed
Jacalz opened this issue Jan 13, 2021 · 3 comments
Closed

High importance button show no hovered state #1785

Jacalz opened this issue Jan 13, 2021 · 3 comments
Labels
bug Something isn't working
Milestone
2.0.x - bugfixes ...

Comments

@Jacalz
Copy link
Member

Jacalz commented Jan 13, 2021

Describe the bug:

Hovering over a button is supposed to indicate that it is hovered, but buttons with HighImportance does not show any hovered indication at all.

To Reproduce:

Steps to reproduce the behaviour:

  1. Go to the form tab in fyne_demo
  2. Fill in an email to make the form valid
  3. Hover over the dismiss button
  4. Hover over submit button and see that it shows no hover

Screenshots:

hover

Device (please complete the following information):

  • OS: Linux
  • Version: 5.10.5
  • Go version: 1.15.6
  • Fyne version: 1.4.3 and develop @ f111a10
@Jacalz Jacalz added the bug Something isn't working label Jan 13, 2021
@AlbinoGeek
Copy link
Contributor

Can this bug be brought into the 2.0.x - bugfixes since binding, storage, animation addition milestone?

[or perhaps I misunderstood the naming of that milestone, and it belongs in the other one?]

Ether way, would love to see this in the next version.

@Jacalz
Copy link
Member Author

Jacalz commented Jan 23, 2021

I agree. It is a very noticeable issue. Added it to that milestone for now :)

@fpabl0 fpabl0 mentioned this issue Feb 22, 2021
Closed
3 tasks
andydotxyz added a commit to andydotxyz/fyne that referenced this issue Feb 23, 2021
@andydotxyz
Add calculations to apply hover over high importance buttons
ffeca25 
Fixes fyne-io#1785
@andydotxyz andydotxyz mentioned this issue Feb 23, 2021
Merged
3 tasks
@andydotxyz
Copy link
Member

Fixed on release/v2.0.x

This was referenced Mar 12, 2021
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
2.0.x - bugfixes since binding, stora...
Development

No branches or pull requests
3 participants
@andydotxyz @AlbinoGeek @Jacalz