Skip to content

style(mobile): video indicator - #4443

Merged
alextran1502 merged 1 commit into
mainfrom
mobile-video-indicator
Oct 11, 2023
Merged

alextran1502 merged 1 commit into
mainfrom
mobile-video-indicator

Conversation

@shenlong-tanwen

Copy link
Copy Markdown
Member

Changes made in the PR

  • Video file indicator has been updated in the mobile app to be more prominent
  • Duration indicator is also updated to be in line with how it looks in the app that must not be named

Screenshot

Before After
Before After

@vercel

vercel Bot commented Oct 11, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
immich ⬜️ Ignored (Inspect) Oct 11, 2023 4:57pm

@alextran1502
alextran1502 merged commit 18fcca2 into main Oct 11, 2023
@alextran1502
alextran1502 deleted the mobile-video-indicator branch October 11, 2023 19:03
@cfitzw

cfitzw commented Oct 12, 2023

Copy link
Copy Markdown
Contributor

Probably worthy of another PR, but something that stands out to me is the cloud state not being visible when there is a white background — see bottom left picture from the example screenshots.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants