Skip to content

Refactor inline style attributes#36652

Merged
lunny merged 6 commits intogo-gitea:mainfrom
silverwind:refactor/replace-inline-styles-with-tailwind
Feb 20, 2026
Merged

Refactor inline style attributes#36652
lunny merged 6 commits intogo-gitea:mainfrom
silverwind:refactor/replace-inline-styles-with-tailwind

Conversation

@silverwind
Copy link
Copy Markdown
Member

@silverwind silverwind commented Feb 17, 2026

This is the result of a full-repo review to look for style attributes that can be replaced with tailwind or other methods. I will manually validate later.

🤖 Generated with Claude Code

Use showElem/hideElem helpers and tw-* classes instead of directly
setting element.style properties, for consistency with the codebase's
Tailwind-first approach.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 17, 2026
@silverwind silverwind added type/refactoring Existing code has been cleaned up. There should be no new functionality. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels Feb 17, 2026
@silverwind silverwind marked this pull request as draft February 17, 2026 04:01
@silverwind silverwind changed the title Replace inline style manipulations with Tailwind classes Replace inline style attributes Feb 17, 2026
@silverwind silverwind changed the title Replace inline style attributes Refactor inline style attributes Feb 17, 2026
Signed-off-by: silverwind <me@silverwind.io>
@silverwind
Copy link
Copy Markdown
Member Author

Tested, works.

@silverwind silverwind marked this pull request as ready for review February 17, 2026 19:38
@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 Feb 17, 2026
@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 Feb 20, 2026
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Feb 20, 2026
@lunny lunny merged commit aedc564 into go-gitea:main Feb 20, 2026
24 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Feb 20, 2026
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Feb 20, 2026
@silverwind silverwind deleted the refactor/replace-inline-styles-with-tailwind branch February 22, 2026 06:31
zjjhot added a commit to zjjhot/gitea that referenced this pull request Feb 23, 2026
* giteaofficial/main: (81 commits)
  Add keyboard shortcuts for repository file and code search (go-gitea#36416)
  Refactor text utility classes to Tailwind CSS (go-gitea#36703)
  Prevent redirect bypasses via backslash-encoded paths (go-gitea#36660)
  Fix force push time-line commit comments of pull request (go-gitea#36653)
  Fix get release draft permission check (go-gitea#36659)
  Move `X_FRAME_OPTIONS` setting from `cors` to `security` section (go-gitea#30256)
  Update JS and PY deps (go-gitea#36708)
  Move jobparser from act repository to Gitea (go-gitea#36699)
  Fix push time bug (go-gitea#36693)
  Add icon to buttons "Close with Comment", "Close Pull Request", "Close Issue" (go-gitea#36654)
  various fixes (go-gitea#36697)
  Add AI Contribution Policy to CONTRIBUTING.md (go-gitea#36651)
  Add some validation on values provided to USER_DISABLED_FEATURES and EXTERNAL_USER_DISABLED_FEATURES (go-gitea#36688)
  Rework e2e tests (go-gitea#36634)
  Remove unused functions (go-gitea#36672)
  Add migration http transport for push/sync mirror lfs (go-gitea#36665)
  Fix track time issue id (go-gitea#36664)
  Refactor inline style attributes (go-gitea#36652)
  Update Nix flake (go-gitea#36679)
  Upgrade gogit to 5.16.5 (go-gitea#36680)
  ...

# Conflicts:
#	web_src/js/features/repo-issue-content.ts
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 skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/refactoring Existing code has been cleaned up. There should be no new functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants