Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

feat(svelte): Change branch and tags page design, including mobile#63914

Merged
fkling merged 4 commits into
mainfrom
fkling/sk/branches-tags-pages
Jul 18, 2024
Merged

feat(svelte): Change branch and tags page design, including mobile#63914
fkling merged 4 commits into
mainfrom
fkling/sk/branches-tags-pages

Conversation

@fkling

@fkling fkling commented Jul 18, 2024

Copy link
Copy Markdown
Contributor

The branches page didn't work well on mobile and neither did the tags page if long tag names were present.

This commit changes how the information is displayed and rendered, especially on mobile.

I also added additional links to each row to make navigating to relevant places easier.

Before After
2024-07-18_15-20 2024-07-18_15-20_1
2024-07-18_15-20_2 2024-07-18_15-20_3
2024-07-18_15-20_4 2024-07-18_15-20_5
2024-07-18_15-21 2024-07-18_15-21_1
2024-07-18_15-21_2 2024-07-18_15-21_3
2024-07-18_15-22 2024-07-18_15-22_1

Test plan

Manual inspection of pages in various screen sizes.

The branches page didn't work well on mobile and neither did the tags
page if long tag names were present.

This commit changes how the information is displayed and rendered,
especially on mobile.

I also added additional links to each row to make navigating to relevant
places easier.
@fkling fkling self-assigned this Jul 18, 2024
@cla-bot cla-bot Bot added the cla-signed label Jul 18, 2024

@peterguy peterguy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Got the walkthrough; looks good! Approving to unblock.

@fkling fkling merged commit 462ff95 into main Jul 18, 2024
@fkling fkling deleted the fkling/sk/branches-tags-pages branch July 18, 2024 19:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants