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

Random bits and pieces 5 #4056

Merged
merged 4 commits into from Aug 26, 2023
Merged

Random bits and pieces 5 #4056

merged 4 commits into from Aug 26, 2023

Conversation

ivarnakken
Copy link
Member

Description

Update some icons to behave better

They will now act as buttons, so you can tab on them.


Give icons and buttons the same "active" look

The translate(-1px) felt less modern and the icons did not have any
active styling.


Handle copy button timeouts better

Less glitchy by clearing out old timeouts when creating new ones.


Fix random bits and pieces, again

  • Remove some use of italic fonts
  • Fix glitchy render on footer
  • Remove some minor bad styling

Result

Less glitchy:

Screen.Recording.2023-07-22.at.16.37.45.mov

You can now open by tabbing, and the "delete row" has a danger look:

Screen.Recording.2023-07-22.at.16.40.08.mov

Testing

  • I have thoroughly tested my changes.

Not really any breaking changes, but button and icon changes were tested on both dark and light theme.

@ivarnakken ivarnakken added enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review bug-fix Pull requests that fix a bug technical-debt Pull requests that reduces technical debt chore Pull requests that does something "boring", yet important, e.g. cleaning up code labels Jul 22, 2023
@ivarnakken ivarnakken requested a review from a team July 22, 2023 14:41
@ivarnakken ivarnakken self-assigned this Jul 22, 2023
@ivarnakken ivarnakken changed the title Update some icons to behave better Random bits and pieces 5 Jul 22, 2023
Copy link
Contributor

@ollfkaih ollfkaih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good like usual, just some nitpicking

app/components/Upload/ImageUpload.tsx Outdated Show resolved Hide resolved
@ivarnakken ivarnakken added the approved Pull requests that have been approved label Jul 22, 2023
They will now act as buttons, meaning you can interact with them easier,
e.g. tabbing through them.
The translate(-1px) felt less modern and the icons did not have any
active styling.
Less glitchy by clearing out old timeouts when creating new ones.
* Remove some use of italic fonts since that's old school
* Fix glitchy render on footer
* Remove some bad styling
@ivarnakken ivarnakken merged commit 0f926ab into master Aug 26, 2023
4 checks passed
@ivarnakken ivarnakken deleted the random-bits-and-pieces-5 branch August 26, 2023 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug chore Pull requests that does something "boring", yet important, e.g. cleaning up code enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review technical-debt Pull requests that reduces technical debt
Projects
None yet
2 participants