Skip to content
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

feat: Enhance text selection menu in reader #204

Merged
merged 3 commits into from
Aug 30, 2024
Merged

Conversation

starry-shivam
Copy link
Member

No description provided.

Co-authored-by: Acclorite <[email protected]>
Signed-off-by: starry-shivam <[email protected]>
@starry-shivam starry-shivam added the enhancement New feature or request label Aug 30, 2024
@starry-shivam starry-shivam merged commit d0b0ec7 into main Aug 30, 2024
1 check passed
@starry-shivam starry-shivam deleted the text-selection branch August 30, 2024 05:19
starry-shivam added a commit that referenced this pull request Aug 30, 2024
* feat: Enhance text selection menu in reader
* Add missing translation strings
* Fix incorrect log message
---------

Signed-off-by: starry-shivam <[email protected]>
Co-authored-by: Acclorite <[email protected]>
@Acclorite
Copy link
Contributor

Just to note. This custom solution is greatly undeveloped, even though it "kinda" works, I'd replace it with new library/solution as fast as there is one.

@starry-shivam
Copy link
Member Author

Just to note. This custom solution is greatly undeveloped, even though it "kinda" works, I'd replace it with new library/solution as fast as there is one.

Yeah, I've read a few threads on Google's Issue Tracker related to that. In most of them, they mentioned that they would implement a proper selection menu, similar to what was available in the XML view system, for Compose in future releases. Well, fast forward two years, and we're still stuck with just one "copy" option. So yeah, it's better than nothing. Thanks for that! I wanted to do something similar but always felt too lazy to look into it, or I was occupied with something else most of the time.

Btw, your reader options/settings also look neat! Don't mind if i pick some of those haha

@Acclorite
Copy link
Contributor

Well, fast forward two years, and we're still stuck with just one "copy" option.

Yeah, that's what annoys me. The topic has minimal movement, same with scrollbar, but for scrollbar there at least is a library..

To make this selection container, I literally spent about a week of reading wiki, stackoverflow and some other sources. This is super inconvenient and had to be fixed and improved long time ago.

Btw, your reader options/settings also look neat!

Really? I plan on reworking them, I feel like your app's settings look much better :D

Don't mind if i pick some of those haha

Well I also am interested in some of your design decisions. I prefer traditional Material3, but your has some interesting decisions ;)

@starry-shivam
Copy link
Member Author

Yeah, that's what annoys me. The topic has minimal movement, same with scrollbar, but for scrollbar there at least is a library..

Exactly, and they keep deprecating existing solutions in favor of half-baked things. For example, they've marked TextToolbar as deprecated without even providing any API to replace it, at least in Compose. I'm not sure about XML since you don't really need custom implementation in XML most of the time.

To make this selection container, I literally spent about a week of reading wiki, stackoverflow and some other sources.

Thanks for your efforts! If I hadn't stumbled upon your code, I wouldn't have worked on this anytime soon due to the similar experience you had to go through.

Really? I plan on reworking them, I feel like your app's settings look much better :D

Yep! I specifically liked settings like line & paragraph spacing and similar things. The settings related to the reader were organized neatly in a bottom sheet.

Well I also am interested in some of your design decisions. I prefer traditional Material3, but your has some interesting decisions ;)

In my previous open-source app (GreenStash), I had mostly used traditional Material 3 design components, so I wanted to try something different with this one. :)

@Acclorite
Copy link
Contributor

Acclorite commented Aug 31, 2024

Yep! I specifically liked settings like line & paragraph spacing and similar things. The settings related to the reader were organized neatly in a bottom sheet.

I try extend customization to it's limits as there is not really much to do with plain text. As for organization I follow one simple rule — all settings should be easily accessible from Settings screen. I try to not make some random options in some screen you would need to search for 10 minutes to find, and seems like it's working as of now.

@starry-shivam
Copy link
Member Author

Yep! I specifically liked settings like line & paragraph spacing and similar things. The settings related to the reader were organized neatly in a bottom sheet.

I try extend customization to it's limits as there is not really much to do with plain text. As for organization I follow one simple rule — all settings should be easily accessible from Settings screen. I try to not make some random options in some screen you would need to search for 10 minutes to find, and seems like it's working as of now.

Yep, I also need to improve and add more options to my reader settings. It's very barebones right now, haha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants