Skip to content

Miscellaneous UI Tweaks #3127

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.

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

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Miscellaneous UI Tweaks #3127

wants to merge 11 commits into from

Conversation

scolsen
Copy link
Contributor

@scolsen scolsen commented Apr 30, 2025

This PR contains a number of small UI tweaks to make the application match the current UX designs, including:

  • Adding a translucent background in the Job selection modal
  • Tweaking the font and icon size of job selection cards
  • Adding a small linear gradient to the bottom of the screen
  • Updating the color of the job selection cancellation action button.

I didn't get precise numbers on sizing dimensions or gradient values I more or less eyeballed it, so I'm happy to adjust if we have precise values.

Screen.Recording.2025-05-08.at.2.01.43.PM.mov

Towards #3078

scolsen added 8 commits April 29, 2025 11:08
Updates the background of the job selection modal to match current designs.
Adds an enum to control coloring of action buttons dependt on a (for now) binary modality of primary or secondary. Updates the color for secondary actions in the job selection modal accordingly.
Adds a subtle linear gradient to the bottom of the home screen to match the latest UX designs.
@auto-assign auto-assign bot requested a review from sufyanAbbasi April 30, 2025 20:04
@scolsen scolsen requested a review from gino-m April 30, 2025 20:04
Copy link
Collaborator

@gino-m gino-m left a comment

Choose a reason for hiding this comment

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

LGTM! A few procedural things:

  • Push code to a new branch in this repo scolsen/3078/ui-tweaks for ex.
  • Run gradle ktfmtFormat commit and push?
  • Run gradle checkCode and fix and errors there
  • Attach screen recording of updated UIs to PR description

@scolsen
Copy link
Contributor Author

scolsen commented May 8, 2025

Done, done, done. What did you mean re:

Push code to a new branch in this repo scolsen/3078/ui-tweaks for ex.

This is in a branch in my ground-android repo -- the branch is called 3078.

@scolsen scolsen requested a review from gino-m May 8, 2025 18:04
@gino-m
Copy link
Collaborator

gino-m commented May 8, 2025

Nice work! One nit: is the gradient the same height as in the designs? To my eye it looks like it starts halfway through the "+" fab. Instead, it should start right below the location lock fab like this:

image

Here are the alpha and color values for reference:

image

android:angle="90"
android:startColor="#601A1C19"
android:centerColor="#301A1C19"
android:endColor="#021A1C19" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

The actual gradient is as follows:

image

Note alpha channel as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants