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
TouchMenu: keep file/exit button on the right #2572
Conversation
145a09a
to
6e84b01
Compare
The after screenshots are a couple of pixels off because I realized I forgot Screenshot's based on self.width - content_width - (math.min(spacing_width, Screen:scaleBySize(20)) * #self.icons *2) I changed it to self.width - content_width - (math.min(spacing_width, Screen:scaleBySize(20)) * #self.icons *2) + Screen:scaleBySize(2), |
frontend/ui/widget/touchmenu.lua
Outdated
@@ -219,6 +219,12 @@ function TouchMenuBar:init() | |||
for i, sep in ipairs(self.icon_seps) do | |||
local current_icon = i == k - 1 or i == k | |||
self.icon_seps[i].style = current_icon and "solid" or "none" | |||
if i == #self.icon_seps - 1 then |
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.
If you like this change then I should add a comment explaining what this does first.
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.
Lol, the screen of the Kobo Mini is so small that the home/exit icon is always on the right, so I won't notice any different.
Looks good. But I do agree the "home" icon is confusion. Several of my friends tried KOReader on my Kobo, the first button they have clicked was the "home". It provides a wrong information as the front page of the app. |
Do we still need this change after #2577 is merged? |
Yes, we do. :)
…On Feb 26, 2017 09:37, "Qingping Hou" ***@***.***> wrote:
Do we still need this one after #2577
<#2577> is merged?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2572 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMYBa4WpQFtRUSxeWfg1uMJvtHgkEtSks5rgTnGgaJpZM4MLVRk>
.
|
Based on the discussion in #2555, @baskerville suggested moving the home icon to the right because it's a uncancellable action. Now that #2577 is merged, this is not the case anymore. So why do we need to move it all the way to the right again? ;P |
Lol
…On Feb 26, 2017 23:30, "Qingping Hou" ***@***.***> wrote:
Based on the discussion in #2555
<#2555>, @baskerville
<https://github.com/baskerville> suggested moving the home icon to the
right because it's a uncancellable action. Now that #2577
<#2577> is merged, this is not
the case anymore. So why do we need to move it all the way to the right
again? ;P
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2572 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMYBRZJO6t11Rq7lLdaLMfbjY2ir1fxks5rgfzrgaJpZM4MLVRk>
.
|
Now, In my opinion, history and open last document look like "search" function, and can be placed in search menu. Book status looks like document feature, can be placed in document menu. Well, this is my personal opinion only. |
I'd put open last document in reader as well for consistency.
But search? Is there any precedent for that in other applications?
…On Feb 27, 2017 09:35, "Hzj_jie" ***@***.***> wrote:
Now,
In file manager, we have history / open last document / version / help and
exit in hamburg.
In reader, we have book status / history / version / help / exit in
hamburg.
In my opinion, history and open last document look like "search" function,
and can be placed in search menu. Book status looks like document feature,
can be placed in document menu.
Then we only have version / help / exit, and maybe power off later in
hamburg, which seems reasonable to be put on the right.
Well, this is my personal opinion only.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2572 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMYBRe3FZPPOhdoQxnhrNafgYPHi8tNks5rgoq7gaJpZM4MLVRk>
.
|
Purely UX-wise, I think it's easier both to find and to tap corner items than something in the middle. I imagine that's also [why] just about everything from the Window start button to GNOME to Android puts hamburger dead left or dead right, not somewhere in the middle to the right end of a bunch of other icons. The sole exception is Opera, where I had to go hunting for the freaking hamburger menu. In their defense I suppose fullscreen is frequently on the bottom right. Anyway, that particular issue doesn't apply to KOReader as long as we keep it right-most, but I can say from working with this change for the past few days that I really regret not having done this a couple of years ago. I wanted to, but I didn't feel comfortable enough at the time, plus I had different priorities (proper translation). I suggest giving it a go for a few days if you're unsure. ;-) |
@Hzj-jie : I agree. It doesn't need to be the "search" tab, but a more generic file "explorer" tag. AFAIK both options are avaliable only from the file manager app, but not from the reader app. The reader app UX seems a good start point here. It has two tabs:the bookmark tab and the --reader.app-- tab, which shows config only avaliable from that app at the left of the screen (those are more specific than the settings or the plugins tab). In file manager app the first icon could be the more related to filemanager as posible IMO If the tabs follow the same logic on all user interfaces it would be a great idea to put those general tabs (plugins, settings & hamburguer) on the right and align left specific tabs for the current view. |
References #2555.
Before:
After: