Skip to content
This repository has been archived by the owner on May 20, 2024. It is now read-only.

❌ WinPaletter crash after replace Segoe UI font #228

Closed
Andrew-Sor opened this issue Apr 8, 2024 · 16 comments
Closed

❌ WinPaletter crash after replace Segoe UI font #228

Andrew-Sor opened this issue Apr 8, 2024 · 16 comments
Assignees
Labels
issue Issue

Comments

@Andrew-Sor
Copy link

Andrew-Sor commented Apr 8, 2024

❌ Error report

Describe the error

WinPaletter crash after replace Segoe UI font, but the system is working fine.

How to reproduce the error

  • Open WinPaletter
  • Go to "Metrics & Font"
  • Go to font replacement section
  • Replace Segoe UI with any one
  • Apply changes
  • Log out and log in again
  • Open WinPaletter again

WinPaletter Log

//General information
//...........................................................
   Report.Date = "8 апреля 2024 г. 19:48:19";
   OS = "Windows 11, 10.0.22631.3374, 64-bit";
   WinPaletter.Version = "1.0.9.0, Build: Release";
   WinPaletter.Language = "Russian";
   WinPaletter.Debugging = false;

//Error details
//...........................................................
   Exception.message = "Error creating form: Object reference not set to an instance of an object.";
   Exception.type = "System.InvalidOperationException";
   Exception.stack.trace =
   {
      at WinPaletter.Forms.CreateInstance[T](T instance)
      at WinPaletter.Forms.get_MainForm()
      at WinPaletter.User.OnUserSwitch(UserChangeEventArgs e)
      at WinPaletter.User.set_SID(String value)
      at WinPaletter.UserSwitch.Button1_Click(Object sender, EventArgs e)
      at System.Windows.Forms.Control.OnClick(EventArgs e)
      at System.Windows.Forms.Button.OnClick(EventArgs e)
      at WinPaletter.UI.WP.Button.OnClick(EventArgs e)
      at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
      at WinPaletter.UI.WP.Button.OnMouseUp(MouseEventArgs e)
      at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
      at System.Windows.Forms.Control.WndProc(Message& m)
      at System.Windows.Forms.ButtonBase.WndProc(Message& m)
      at System.Windows.Forms.Button.WndProc(Message& m)
      at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
      at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
      at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
   };
   Exception.target.void_function = "WinPaletter.CreateInstance()";
   Exception.assembly = "WinPaletter, Version=1.0.9.0, Culture=neutral, PublicKeyToken=null";
   Exception.assembly.file = "D:\\Downloads🔽\\WinPaletter\\WinPaletter.exe";
   Exception.HRESULT = -2146233079;
   Inner.exception.message = "Object reference not set to an instance of an object.";
   Exception.type = "System.NullReferenceException";
   Inner.exception.stack.trace =
   {
      at WinPaletter.Tabs.TitlebarExtender.updateBackDrop()
      at WinPaletter.Tabs.TitlebarExtender.OnSizeChanged(EventArgs e)
      at System.Windows.Forms.Control.UpdateBounds(Int32 x, Int32 y, Int32 width, Int32 height, Int32 clientWidth, Int32 clientHeight)
      at System.Windows.Forms.Control.UpdateBounds(Int32 x, Int32 y, Int32 width, Int32 height)
      at System.Windows.Forms.Control.SetBoundsCore(Int32 x, Int32 y, Int32 width, Int32 height, BoundsSpecified specified)
      at System.Windows.Forms.Control.ScaleControl(SizeF factor, BoundsSpecified specified)
      at System.Windows.Forms.ScrollableControl.ScaleControl(SizeF factor, BoundsSpecified specified)
      at System.Windows.Forms.Control.ScaleControl(SizeF includedFactor, SizeF excludedFactor, Control requestingControl)
      at System.Windows.Forms.Control.Scale(SizeF includedFactor, SizeF excludedFactor, Control requestingControl)
      at System.Windows.Forms.ContainerControl.Scale(SizeF includedFactor, SizeF excludedFactor, Control requestingControl)
      at System.Windows.Forms.ContainerControl.OnChildLayoutResuming(Control child, Boolean performLayout)
      at System.Windows.Forms.ContainerControl.OnLayoutResuming(Boolean performLayout)
      at System.Windows.Forms.Control.ResumeLayout(Boolean performLayout)
      at System.Windows.Forms.ContainerControl.ResumeAllLayout(Control start, Boolean performLayout)
      at System.Windows.Forms.ContainerControl.OnFontChanged(EventArgs e)
      at System.Windows.Forms.Form.OnFontChanged(EventArgs e)
      at System.Windows.Forms.Control.set_Font(Font value)
      at WinPaletter.MainForm.InitializeComponent()
      at WinPaletter.MainForm..ctor()
   };
   Inner.exception.target.void_function = "WinPaletter.updateBackDrop()";
   Inner.exception.assembly = "WinPaletter, Version=1.0.9.0, Culture=neutral, PublicKeyToken=null";
   Inner.exception.assembly.file = "D:\\Downloads🔽\\WinPaletter\\WinPaletter.exe";
   Inner.exception.HRESULT = -2147467261;

Screenshots/Screen records

  • Don't add screenshot to exception error dialog if you already wrote WinPaletter Log.

Additional context

There is enough information above

@Andrew-Sor Andrew-Sor added the issue Issue label Apr 8, 2024
@Andrew-Sor Andrew-Sor changed the title WinPaletter crash after replace Segoe UI font ❌ WinPaletter crash after replace Segoe UI font Apr 8, 2024
@Abdelrhman-AK
Copy link
Owner

Abdelrhman-AK commented Apr 8, 2024

It is not relevant to the action of replacing Segoe UI. It is relevant to the application's UI. You can safely ignore this error.

If I work on version 1.0.9.1, I'll fix it for sure. Please keep this issue open.

@Abdelrhman-AK
Copy link
Owner

And please tell me if you're disabling transparency effect on Windows or using another theme (visual styles) for Windows.

@Andrew-Sor
Copy link
Author

Well I'm using the MacOS theme and I didn't turn off the transparency effect

@Andrew-Sor
Copy link
Author

Andrew-Sor commented Apr 8, 2024

It is not relevant to the action of replacing Segoe UI. It is relevant to the application's UI.

I don’t know, but I changed all sorts of settings and applied them several times, restarting WinPaletter itself. Today I only replaced Segoe UI and got this...
And I believe that the system is working fine because in most places the font has reset (for example, in the headings it’s not even bold, although I set it up yesterday)

You can safely ignore this error.

Unfortunately, after that it gives the error again, but if you ignore it this time, it will close

@Abdelrhman-AK
Copy link
Owner

When I work on the next version, I'll fix it for sure.

@Abdelrhman-AK
Copy link
Owner

Abdelrhman-AK commented Apr 8, 2024

Can you use both WinPaletter-DEBUG.exe with WinPaletter.pdb so that I can trace the issue. I can't reproduce it at all.
Download both from releases if you don't have them.

I reproduced and fixed it. Please wait for a new update, which will likely be the final one for me as I won't have time after a few days.

@Andrew-Sor
Copy link
Author

Andrew-Sor commented Apr 9, 2024

Can you use both WinPaletter-DEBUG.exe with WinPaletter.pdb so that I can trace the issue. I can't reproduce it at all.

I thought about that too... But they also gave an error...

I reproduced and fixed it. Please wait for a new update, which will likely be the final one for me as I won't have time after a few days.

Excellent thank you! I'll be waiting

@Andrew-Sor
Copy link
Author

By the way, the old version works
image

@Abdelrhman-AK
Copy link
Owner

Abdelrhman-AK commented Apr 9, 2024

There is something wrong I have fixed, it is in the tabs UI that wasn't implemented in older versions.

@Andrew-Sor
Copy link
Author

old version

Using it, I was able to reset the Segoe UI replacement and the new version now works fine too

@Andrew-Sor
Copy link
Author

Andrew-Sor commented Apr 9, 2024

By the way, I also wanted to ask about the menu animation not working

2024-04-09-20-30-15.mp4

Or is it because I have Windows 11?

@Abdelrhman-AK
Copy link
Owner

Abdelrhman-AK commented Apr 9, 2024

That is because you have Windows 11. If you enabled classic context menu or used classic context menu in a Win32 app (for example: Registry Editor), you will notice the effect.

image

@Andrew-Sor
Copy link
Author

Andrew-Sor commented Apr 9, 2024

That is because you have Windows 11. If you enabled classic context menu or used classic context menu in a Win32 app (for example: Registry Editor), you will notice the effect.

It's clear. Thanks, but I like the new design better

@Abdelrhman-AK
Copy link
Owner

I like the new design better

Absolutely! Feel free to personalize your Windows as you like.

@Andrew-Sor
Copy link
Author

Thank you! Now everything works great!

@Abdelrhman-AK
Copy link
Owner

You're welcome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue Issue
Projects
None yet
Development

No branches or pull requests

2 participants