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.

Sign up for GitHub

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

Jump to bottom

Various UI tweaks 2 #2677

Merged
merged 17 commits into from
Feb 19, 2025
Merged

Various UI tweaks 2 #2677

merged 17 commits into from
Feb 19, 2025

Conversation

insomnious
Copy link
Collaborator

@insomnious insomnious commented Feb 19, 2025 •
edited
Loading

Completed:

  • Moved Give Feedback to bottom bar
  • Added icons to topbar menus and adjusted text
  • New welcome message modal
  • Fix for large spinner in collection download action column

Above has been checked with @captainsandypants

  • URL for Give Feedback button will be changed closer to release (@captainsandypants)
  • Welcome screen styles are inline, more work to do over the coming few days and may well pull them out into a separate file then

Part of #2652

@insomnious insomnious requested a review from a team February 19, 2025 11:06
@insomnious insomnious self-assigned this Feb 19, 2025
@insomnious insomnious marked this pull request as ready for review February 19, 2025 11:25
@github-actions github-actions bot added the status-needs-rebase Set by CI do not remove label Feb 19, 2025
@github-actions GitHub Actions
Copy link
Contributor

This PR conflicts with main. You need to rebase the PR before it can be merged.

@github-actions github-actions bot removed the status-needs-rebase Set by CI do not remove label Feb 19, 2025
@github-actions GitHub Actions
Copy link
Contributor

This PR doesn't conflict with main anymore. It can be merged after all status checks have passed and it has been reviewed.

@insomnious insomnious merged commit 062cbeb into main Feb 19, 2025
11 checks passed
@insomnious insomnious deleted the various-ui-tweaks-2 branch February 19, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Al12rs Al12rs Al12rs approved these changes

Assignees

@insomnious insomnious

Labels
Dev UI
Projects
MVP
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@insomnious @Al12rs