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

fix: fix tab bar color not properly applying #85

Merged
merged 4 commits into from
Oct 9, 2024

Conversation

saeedbashir
Copy link

@saeedbashir saeedbashir commented Oct 7, 2024

LEARNER-10246

Before fix After fix
nav_bar_issue.mov
nav_bar.mov

@saeedbashir saeedbashir changed the title fix: fix tar bar color not properly applying fix: fix tab bar color not properly applying Oct 7, 2024
@rnr
Copy link
Collaborator

rnr commented Oct 7, 2024

@saeedbashir This fix looks good for me!
One thing I found when TabBar has good color now - Learn icon for selected and unselected states has different height and a bit jumps when select/deselect tab
Screenshot 2024-10-07 at 12 10 26
Could you please look on this too?
Thank you

@shafqat-muneer
Copy link

Changes LGTM!
We can merge after addressing @rnr's comment.

@saeedbashir
Copy link
Author

@saeedbashir This fix looks good for me!
One thing I found when TabBar has good color now - Learn icon for selected and unselected states has different height and a bit jumps when select/deselect tab

@rnr Done! There was a change for the discover icon after the PR merge, I've also addressed that.

@saeedbashir saeedbashir merged commit e3ce9bd into 2U/develop Oct 9, 2024
3 checks passed
@saeedbashir saeedbashir deleted the saeed/LEARNER-10246 branch October 9, 2024 04:21
rnr pushed a commit that referenced this pull request Dec 13, 2024
* fix: fix tar bar color not properly applying
* chore: parity with android for discover icon
rnr added a commit that referenced this pull request Dec 17, 2024
* feat: banner when register with already linked social account (#54)

* fix: fix lint warnings

* chore: added banner when register with already linked social account

* style: changed text color

---------

Co-authored-by: Anton Yarmolenko <[email protected]>

* chore: fix after merge

* chore: update tab bar icons for selected unselected states (#81)

* chore: Elm theme improvements on Profile (#82)

* chore: fix after merge

* fix: fix tab bar color not properly applying (#85)

* fix: fix tar bar color not properly applying
* chore: parity with android for discover icon

* chore: added missed colors

---------

Co-authored-by: Anton Yarmolenko <[email protected]>
Co-authored-by: Saeed Bashir <[email protected]>
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