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,
editorFontSizeinViewCommandsis not updating after increase or decrease. For example, ifeditorFontSizevalue is 12, after increasing it will be 13 and it will be rendered buteditorFontSizevalue inViewCommandswon't be updated soeditorFontSizeremain 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, justeditorFontSizedid 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.wrappedValueinAppSettings. 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