32
41
Fork
You've already forked website
52

revamp header and add "donate" link #473

Merged
earl-warren merged 2 commits from caesar/forgejo-website:header into main 2024年08月06日 07:34:38 +02:00
Member
Copy link
  • cleans up the header code to reduce duplication (including reducing clutter of Tailwind class names)
  • fixes some styling inconsistencies, especially highlight colours in light mode
  • adds a "Donate" link (closes #462 – see also #472)

Before:

After:

- cleans up the header code to reduce duplication (including reducing clutter of Tailwind class names) - fixes some styling inconsistencies, especially highlight colours in light mode - adds a "Donate" link (closes #462 – see also #472) Before: ![](/attachments/cdbafcdd-ec2e-4a15-bf7b-92e7139c71bb) After: ![](/attachments/d4fb636e-115d-4e00-93f5-305189d366de)
Collaborator
Copy link

Preview ready: https://forgejo.codeberg.page/@pull_473/

Preview ready: https://forgejo.codeberg.page/@pull_473/
caesar force-pushed header from 1d617725a5
All checks were successful
pr / preview (pull_request) Successful in 41s
to ce21aec80e
All checks were successful
pr / preview (pull_request) Successful in 41s
2024年08月06日 01:06:51 +02:00 Compare
Contributor
Copy link

@caesar I see your commits are meaningful and won't squash them. Please let me know if you would like the merge to be done differently.

@caesar I see your commits are meaningful and won't squash them. Please let me know if you would like the merge to be done differently.
Author
Member
Copy link

@earl-warren thanks, I intended for them not to be squashed.

@earl-warren thanks, I intended for them not to be squashed.
Sign in to join this conversation.
Labels
Clear labels
404
Broken links or missing content
Accessibility
Accessibility
Blog post
Documentation
Forgejo Documentation
Internationalisation
i18n and l10n
User research - Accessibility
Requires input about accessibility features, likely involves user testing.
User research - Blocked
Do not pick as-is! We are happy if you can help, but please coordinate with ongoing redesign in this area.
User research - Community
Community features, such as discovering other people's work or otherwise feeling welcome on a Forgejo instance.
User research - Config (instance)
Instance-wide configuration, authentication and other admin-only needs.
User research - Errors
How to deal with errors in the application and write helpful error messages.
User research - Filters
How filter and search is being worked with.
User research - Future backlog
The issue might be inspiring for future design work.
User research - Git workflow
AGit, fork-based and new Git workflow, PR creation etc
User research - Labels
Active research about Labels
User research - Moderation
Moderation Featuers for Admins are undergoing active User Research
User research - Needs input
Use this label to let the User Research team know their input is requested.
User research - Notifications/Dashboard
Research on how users should know what to do next.
User research - Rendering
Text rendering, markup languages etc
User research - Repo creation
Active research about the New Repo dialog.
User research - Repo units
The repo sections, disabling them and the "Add more" button.
User research - Security
User research - Settings (in-app)
How to structure in-app settings in the future?
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo/website!473
Reference in a new issue
forgejo/website
No description provided.
Delete branch "caesar/forgejo-website:header"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?