Skip to content

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

Draft
nguyeenvy wants to merge 2 commits into
masterfrom
dev/vyng/Lightweight-Styling-NavigationView
Draft

feat(navigationview): Add lightweight styling to navigation view styles#1057
nguyeenvy wants to merge 2 commits into
masterfrom
dev/vyng/Lightweight-Styling-NavigationView

Conversation

@nguyeenvy

Copy link
Copy Markdown
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 requested review from Xiaoy312 and agneszitte July 12, 2023 19:16
@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
Copy Markdown
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

silviuo commented Sep 26, 2023

Copy link
Copy Markdown
Contributor

@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

kazo0 commented Sep 26, 2023

Copy link
Copy Markdown
Collaborator

@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

silviuo commented Sep 28, 2023

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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.

4 participants