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

Feature/add hover texts #133

Merged
merged 3 commits into from
Jan 16, 2024
Merged

Conversation

A-K-O-R-A
Copy link
Contributor

Description

This PR adds title attributes to the shortcut buttons in the popup menu.

image

References

I created the PR because new users might not be familiar with all services and their icons

Type of change

  • Bug fix (non-breaking change which fixes a bug)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that might cause existing functionality to not work as expected)

Testing

We have 1500+ Users. Please test your changes thoroughly.

  • Tested my changes on Firefox
  • Tested my changes on Chromium-Based-Browsers
  • Successfully run npm run test locally

Additional Information

I also removed the closing </div> tag on line 131 as there was no opening tag for it.

@A-K-O-R-A A-K-O-R-A requested a review from a team as a code owner January 16, 2024 13:33
@A-K-O-R-A A-K-O-R-A requested a review from cornzz January 16, 2024 13:33
src/freshContent/popup/popup.html Outdated Show resolved Hide resolved
src/freshContent/popup/popup.html Outdated Show resolved Hide resolved
@OliEfr
Copy link
Member

OliEfr commented Jan 16, 2024

Thank you very much - that is an awesome improvement!

@OliEfr
Copy link
Member

OliEfr commented Jan 16, 2024

Merci

@OliEfr OliEfr merged commit bde0e56 into TUfast-TUD:develop Jan 16, 2024
1 check passed
@A-K-O-R-A A-K-O-R-A deleted the feature/add_hover_texts branch January 16, 2024 15:04
OliEfr added a commit that referenced this pull request Jan 23, 2024
* fix _execute_action for ff

* Fix npm commands (#132)

Thanks @A-K-O-R-A for noticing this bug and reporting it! Have a good day.

* Feature/add hover texts (#133)

* Add hover texts for the popup icons

* Extend Informatik discord hover text

* Correct spelling

* Add OTP support (#131)

* Add first classes for otp saving and generation

* Typo

* OTP saving complete

* Better OTP saving

* OTP filling

* Add recovery codes to prompt

* Eslint

---------

Co-authored-by: Daniel Kluge <daniel-git@c0ntroller.de>
Co-authored-by: Maf <65976562+A-K-O-R-A@users.noreply.github.com>
Co-authored-by: Daniel <43993811+C0ntroller@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants