You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
🐞 Noticeable delay occurs when changing Settings #1247
Changing any setting causes a slight delay before the UI is updated to reflect the change. This is problematic when interacting with a stepper control when it is common to click several times consecutively or a text field when it is common to type characters consecutively. The UI doesn't have time to respond resulting in a sluggish UI.
To Reproduce
Open Settings
Navigate to the Text Editing page
In the Font Size field, click the up arrow to increase five times in a row
Notice that the UI does not update immediately resulting in it feeling sluggish to interact with
Expected Behavior
The UI should respond to user action immediately.
Version Information
CodeEdit: 0.0.3-dev (32)
Additional Context
This might be because it is writing to settings.json in our Application Support folder. I think it waits until it has been updated (not sure, just a theory). I think we should be handling this more optimistically and not be so dependent on writing to the file.
We've discovered the root of the problem. The MonospacedFontPicker is causing this. When this is removed, performance is fine when changing a font size. If the user has a lot of fonts installed, for whatever reason it effects other fields in the form. If the Other Fonts picker is taken out of this view, everything seems to be fine.
Things we might try:
Only include the other fonts Picker when the monospaced fonts picker menu is open
Explore if there is a way to lazy-load these menu items
Activity
Fixed some performance issues (#182)
austincondiff commentedon May 20, 2023
We've discovered the root of the problem. The MonospacedFontPicker is causing this. When this is removed, performance is fine when changing a font size. If the user has a lot of fonts installed, for whatever reason it effects other fields in the form. If the Other Fonts picker is taken out of this view, everything seems to be fine.
Things we might try:
austincondiff commentedon Apr 19, 2025
Changing settings in the current version is much quicker than in the screen recording. Closing.