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

fixes hover issues #351

Merged
merged 2 commits into from
Oct 15, 2022
Merged

fixes hover issues #351

merged 2 commits into from
Oct 15, 2022

Conversation

N6REJ
Copy link

@N6REJ N6REJ commented Oct 13, 2022

Pull Request for Issue # .
#340

Summary of Changes

By default badges do not have hover styling

Examples 
Badges scale to match the size of the immediate parent element by using relative font sizing and em units. As of v5, badges no longer have focus or hover styles for links.

so in order to do so we have the choice of creating a new .css file and adding our own stylings or use the inbuilt bs5 button behavior. I chose the latter.
The effect is subtle but this is due to how atum handles things. This issue would be best for @JaT

Testing Instructions

install patch tester.
apply patch
notice the subtle shift in background color & flicker on hover
image

@roland-d roland-d merged commit 8b9a3e8 into joomla-extensions:master Oct 15, 2022
@roland-d
Copy link
Collaborator

Thanks

@roland-d roland-d added this to the 4.2.2 milestone Oct 15, 2022
@N6REJ N6REJ deleted the 340 branch October 16, 2022 00:05
@N6REJ
Copy link
Author

N6REJ commented Oct 16, 2022

@roland-d would you tag it for hacktoberfest please

@roland-d
Copy link
Collaborator

@N6REJ Done

@N6REJ
Copy link
Author

N6REJ commented Oct 17, 2022

@roland-d nope, didn't work. I think its the wrong label.
image

@roland-d
Copy link
Collaborator

@N6REJ How about now?

@N6REJ
Copy link
Author

N6REJ commented Oct 19, 2022

I think so, yes.. ty..
image

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.

2 participants