Fixed delay in synchronization of font size increment and decrement #1609
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR should fix #1396 but I am not sure.
My understanding is that,
editorFontSize
inViewCommands
is not updating after increase or decrease. For example, ifeditorFontSize
value is 12, after increasing it will be 13 and it will be rendered buteditorFontSize
value inViewCommands
won't be updated soeditorFontSize
remain same (in this case 12 and its increased value always will be 13, so there will be no change to see) but it changed and it is 13, justeditorFontSize
did not get the updated value. The same goes forterminalFontSize
.To fix it, I tried some ways (using binding etc.) but only working way I found is returning
Settings.shared.preferences[keyPath: keyPath]
from getters instead ofsettings.wrappedValue
inAppSettings
. But I am not sure if it is the correct solution. However, there is no broken part I can see so it can be the solution.I also improved readability of the code and removed deprecated code.
Related Issues
Checklist
Screenshots
Font increase and decrease works
screenrecord.mov