Skip to content

UI: add hover background to table rows in user and repo admin page #35072

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

Merged
merged 1 commit into from
Jul 14, 2025

Conversation

BLumia
Copy link
Member

@BLumia BLumia commented Jul 14, 2025

This is a small QoL change to make the admin easier to do their management job. This makes knowing which row is the correct row to operate easier by simply checking the hover background, especially when they have a wide screen.

Sample:

image

The 2nd row is hovered, so before the admin clicks the "delete" button, they can know Magical8bitPlug2 is the one that the "delete" operation will be applied to.

This patch only added the hover background (selectable) to the repo admin and user admin list page. Other tables can be added later.

Note: I'm not sure if my table.css change is sane, feel free to modify this PR directly if needed/preferred)

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 14, 2025
@github-actions github-actions bot added modifies/templates This PR modifies the template files modifies/frontend labels Jul 14, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 14, 2025
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 14, 2025
@techknowlogick techknowlogick merged commit ece0ce6 into go-gitea:main Jul 14, 2025
26 checks passed
@GiteaBot GiteaBot added this to the 1.25.0 milestone Jul 14, 2025
@techknowlogick techknowlogick added type/enhancement An improvement of existing functionality topic/ui Change the appearance of the Gitea UI labels Jul 14, 2025
@BLumia BLumia deleted the selectable branch July 14, 2025 15:09
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jul 16, 2025
* giteaofficial/main:
  Send email on Workflow Run Success/Failure (go-gitea#34982)
  [skip ci] Updated translations via Crowdin
  Replace `poetry` with `uv` (go-gitea#35084)
  nix flake update (go-gitea#35085)
  Use monospace font in PR command line instructions (go-gitea#35074)
  Add gitignore rules to exclude LLM instruction files (go-gitea#35076)
  [skip ci] Updated translations via Crowdin
  Fix form property assignment edge case (go-gitea#35073)
  Improve submodule relative path handling (go-gitea#35056)
  Fixed all grammatical errors in locale_en-US.ini (go-gitea#35053)
  UI: add hover background to table rows in user and repo admin page (go-gitea#35072)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend modifies/templates This PR modifies the template files topic/ui Change the appearance of the Gitea UI type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants