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

[on hold] Implement roles for volume buttons: none, text size, send history #297

Closed
wants to merge 1 commit into from

Conversation

zopieux
Copy link
Collaborator

@zopieux zopieux commented Jul 22, 2016

Keeps the default setting to "text size" for backwards compatibility. This implements feature request #281.

Change summary:

  • preferences: replaces volume button for text-size checkbox with a dropdown with three choices: do nothing, text size, navigate send history
  • change implementation in BufferFragment to account for the new setting

Note that when the user starts to type something in uiInput then decides to navigate history, this draft is stored so they can come back to it by pressing volume down all the way.

@oakkitten
Copy link
Collaborator

this works, but has a couple of problems (perhaps insignificant)

first, we only keep 3 fragments (current one and 2 to the sides of the current) and when you scroll farther they get destroyed, and this will screw things a bit; also there some other ways to destroy a fragment

second, the code could perhaps be more eloquent? maybe only store current input when you push up for the first time and get rid of uiInputSetProgrammatically and move more things to one place. also lines 475-476 get duplicated, and stuff like that

i don't mind this otherwise, me

@zopieux
Copy link
Collaborator Author

zopieux commented Oct 19, 2016

@oakkitten

first, we only keep 3 fragments (current one and 2 to the sides of the current) and when you scroll farther they get destroyed

Where would you suggest I store my state? Would it be okay/needed to persist it as app data (on disk)?

second, the code could perhaps be more eloquent? maybe only store current input when you push up for the first time and get rid of uiInputSetProgrammatically and move more things to one place. also lines 475-476 get duplicated, and stuff like that

I'll review this kind of stuff once I know where to put the state so it does not get destroyed with the fragment.

Keeps the default setting to "text size" for backwards compatibility.
@oakkitten
Copy link
Collaborator

Where would you suggest I store my state?

if you want to store it—not like we can't live without it, after all it's option—

one way is inside the fragment (some bundle is passed in onCreate)
another is making some kind of manual storage (this will help preserve state when user presses back)
another is storing everything in BufferList which is static
another is getting rid of the fragments and and all the stupid clutches that are needed for fragments and just put Views into ViewPager...

another thing, if there's any storage per buffer we could store more things, like current position in chat (sucks when you scroll up in chat and click a link and then get back and it's on the bottom again) or something else

perhaps join #weechat-android on freenode?

@zopieux zopieux changed the title Implement roles for volume buttons: none, text size, send history [on hold] Implement roles for volume buttons: none, text size, send history Oct 21, 2016
@oakkitten oakkitten force-pushed the master branch 2 times, most recently from 3789f56 to 199d6cc Compare June 22, 2021 02:31
@zopieux
Copy link
Collaborator Author

zopieux commented Nov 26, 2022

Superseded by #554.

@zopieux zopieux closed this Nov 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants