-
Notifications
You must be signed in to change notification settings - Fork 359
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add text case changing functionality (Fixes #2390) #2426
base: master
Are you sure you want to change the base?
Conversation
…tor, addressing issue gsantner#2390. Users can now toggle case, switch case, capitalize words, and capitalize sentences directly from the Edit menu.
app/src/main/res/values/strings.xml
Outdated
@@ -477,4 +477,9 @@ work. If not, see <https://creativecommons.org/publicdomain/zero/1.0/>. | |||
<string name="wont_save_min_length">Files with fewer than %d characters not saved automatically in order to prevent data loss.</string> | |||
<string name="advanced_filtering_help"><![CDATA[<small><a href="https://www.github.com/gsantner/markor/discussions/1940">Advanced filtering syntax</a></small>]]></string> | |||
<string name="filter">Filter</string> | |||
<string name="text_case">Text Case</string> | |||
<string name="toggle_case">Toggle Case</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks,
Suggestion, put a example at each like
AbC -> aBc
Abc -> ABC
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, in which file should I add the examples? Should I add them into TextCasingUtils.java, DocumentEditAndViewFragment.java, or HighlightingEditor.java?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Display text (strings.xml)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I see, thank you. I added an example to the display text of each text case now.
Improvement to the Edit menu to include ability to change case. Fixes issue #2390 "Change text case".
This pull request introduces the following changes:
These changes allow users to perform the following text case operations from the Edit menu:
The new functionality respects the user's text selection, applying changes only to the selected text if a selection is active, or to the entire document if no text is selected. Please review and let me know if any changes or improvements are needed.