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.

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

Use activeComponentBG for Workspace menu items #4314

Merged
merged 1 commit into from
Aug 2, 2021

Conversation

timszot
Copy link
Contributor

@timszot timszot commented Jul 29, 2021

Details

Uses a darker grey background for active menu items in the Workspace LHN (Left Hand Nav)

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/171672

Tests

  1. Log into account that has a Workspace setup
  2. Click on your profile pic to bring up settings nav
  3. Click into your setup Workspace to see your options
  4. Click on the menu items and confirm darker grey background on the active item

QA Steps

Same as tests steps above.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

test1.mov

Mobile Web

Desktop

iOS

Android

@timszot timszot requested a review from a team July 29, 2021 21:53
@timszot timszot self-assigned this Jul 29, 2021
@MelvinBot MelvinBot requested review from Gonals and removed request for a team July 29, 2021 21:53
@Gonals Gonals merged commit 609826f into main Aug 2, 2021
@Gonals Gonals deleted the timszot_workspaceLHNHighlights branch August 2, 2021 14:10
@OSBotify
Copy link
Contributor

OSBotify commented Aug 2, 2021

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

OSBotify commented Aug 6, 2021

🚀 Deployed to staging in version: 1.0.82-8🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @francoisl in version: 1.0.83-1 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 failure ❌
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

3 participants