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

feat(navigationview): Add lightweight styling to navigation view styles #1057

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nguyeenvy
Copy link
Contributor

GitHub Issue: https://github.com/unoplatform/uno.themes-private/issues/5

PR Type

What kind of change does this PR introduce?

  • Feature

Description

PR Checklist

  • Tested UWP
  • Tested iOS
  • Tested Android
  • Tested WASM
  • Tested MacOS
  • Contains No breaking changes

Other information

Internal Issue (If applicable):

@nguyeenvy nguyeenvy self-assigned this Jul 12, 2023
@nguyeenvy nguyeenvy force-pushed the dev/vyng/Lightweight-Styling-NavigationView branch 3 times, most recently from 6dc2200 to 912299f Compare July 24, 2023 20:55
@nguyeenvy nguyeenvy marked this pull request as draft July 26, 2023 19:10
@nguyeenvy
Copy link
Contributor Author

Waiting for PR #1106 to merge before updating this PR.

@nguyeenvy nguyeenvy marked this pull request as ready for review August 10, 2023 16:17
@agneszitte agneszitte marked this pull request as draft August 11, 2023 16:13
@silviuo silviuo self-assigned this Sep 26, 2023
@silviuo
Copy link
Contributor

silviuo commented Sep 26, 2023

@kazo0
Is this ready for review? I don't see any comments on the changed files and pr #1106 was merged. Do you know what's missing here? Thanks!

@kazo0
Copy link
Collaborator

kazo0 commented Sep 26, 2023

@silviuo I kept this one for last because I knew it'd be big and then I forgot :) but there will be comments from what I see here. Will look it over now

@kazo0 kazo0 marked this pull request as ready for review September 26, 2023 20:07
@silviuo silviuo force-pushed the dev/vyng/Lightweight-Styling-NavigationView branch from f3b4c82 to e0f6d96 Compare September 28, 2023 15:49
@silviuo
Copy link
Contributor

silviuo commented Sep 28, 2023

I rebased off latest master and refactored resource values to keys

@agneszitte agneszitte marked this pull request as draft October 6, 2023 17:56
@agneszitte
Copy link
Contributor

Let's revisit if this PR is still needed after @Xiaoy312 current work with a script

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.

None yet

4 participants