Skip to content

Update to WinUI 2.7#976

Open
BobbyESP wants to merge 7 commits into
0x7c13:masterfrom
BobbyESP:master
Open

Update to WinUI 2.7#976
BobbyESP wants to merge 7 commits into
0x7c13:masterfrom
BobbyESP:master

Conversation

@BobbyESP

@BobbyESP BobbyESP commented Oct 30, 2021

Copy link
Copy Markdown

App style update

PR Type

What kind of change does this PR introduce?

  • Updated most of the components of the app to WinUI 2.7

Bugs known

  • Settings NavigationView doesn't load the first NavigationItem

Captura de pantalla 2021-10-30 182615

  • Some strange errors when opening the app in App.xaml.cs file, line 50 related with AppCenter (never heard about that)

dependabot Bot and others added 5 commits October 30, 2021 18:08
nuget-deps: bump Microsoft.AppCenter.Crashes in /src/Notepads (0x7c13#959)

Bumps Microsoft.AppCenter.Crashes from 4.1.0 to 4.4.0.

---
updated-dependencies:
- dependency-name: Microsoft.AppCenter.Crashes
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jiaqi Liu <jackil@microsoft.com>
@BobbyESP

Copy link
Copy Markdown
Author

The errors in App.xaml.cs:
Captura de pantalla 2021-10-30 183705
Captura de pantalla 2021-10-30 183749
Captura de pantalla 2021-10-30 183730

@BobbyESP

Copy link
Copy Markdown
Author

Ah yes, forgot to tell that the Navigation Item that doesn't open when you open settings can be opened by clicking on it. Just fails when you open the settings.

@BobbyESP BobbyESP changed the title Update to Windows 11 style, WinUI 3.0 Update to WinUI 2.7 Oct 30, 2021
@0x7c13

0x7c13 commented Oct 31, 2021

Copy link
Copy Markdown
Owner

I will be working on the visual update by EOY. I will wait for WinUI 2.x to get mature before doing so.

@BobbyESP

Copy link
Copy Markdown
Author

It's ok. If you need some help, tell me. WinUI 3 by the moment isn't avaliable for UWP or it's very unestable, so better wait.

@0x5bfa

0x5bfa commented Dec 25, 2021

Copy link
Copy Markdown

Delete all custom style and update UI:
image

@0x5bfa

0x5bfa commented Mar 5, 2022

Copy link
Copy Markdown

@BobbyESP Hai. how have you seen:)

Settings NavigationView doesn't load the first NavigationItem

FYI, It's a not bug but specification. so, need to load the page in NaviView_Loaded() or Page_Loaded()

Instead, use the IsSelected flag to receive change events on ItemSelected. but this app uses ItemInvoked() this do not trigger selection changing event from IsSelected = true.

https://github.com/JasonStein/Notepads/blob/master/src/Notepads/Views/Settings/SettingsPage.xaml#L18

@BobbyESP

BobbyESP commented Mar 5, 2022

Copy link
Copy Markdown
Author

Ohh, i see!

@0x7c13

0x7c13 commented Mar 14, 2022

Copy link
Copy Markdown
Owner

I am waiting for WinUI v2.8 stable release before looking into upgrading the UX. Are there any issues you find with v2.8 prerelease?

@0x5bfa

0x5bfa commented Mar 15, 2022

Copy link
Copy Markdown

ah. I don't have the problem I saw in WinUI prerelease in UI Control, but I found a lot in WerbView2 regardless of whether you are using WebView2.

Also, due to that issue, projects that download WinUI 2.8 prerelease may fail in store submissions.
In fact, I came across that error of win cert kit.

It may be a big help, why not use WinUI2.7 stable ver?

@0x5bfa

0x5bfa commented Mar 15, 2022

Copy link
Copy Markdown

And when updating to WinUI 2.7 or later, the IsSelect property does not trigger the ItemInvoked event. It still triggers SelectionChanged, so you need to navigate to the initial page when loading the settings page. You will find it during the migration.

@0x7c13

0x7c13 commented Mar 15, 2022

Copy link
Copy Markdown
Owner

There are some other UI issues bringing by the v2.7 release, I cannot remmeber the exact but it has been disscussed earlier. The rolling speed of the WinUI 2 and 3 releases are much slower than my expectation:(

@KParthSingh

Copy link
Copy Markdown

Guys It has been over two years now. When will this UI update take place?

@KParthSingh

Copy link
Copy Markdown

@0x5bfa

0x5bfa commented Feb 20, 2024

Copy link
Copy Markdown

The owner hesitates to move to WinUI2.8 or 3.0 because it would bring more WinUI-side issues.

@KParthSingh

Copy link
Copy Markdown

Damn.

@emvaized

emvaized commented Mar 19, 2024

Copy link
Copy Markdown

WinUI-side issues

Are these issues are still relevant as for today? WinUI v2.8 stable release came out quite a long time ago.

@0x5bfa

0x5bfa commented Mar 19, 2024

Copy link
Copy Markdown

I said the same thing to them. lol.

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.

5 participants