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

reorganize touch menu items #2562

Closed
houqp opened this issue Feb 22, 2017 · 25 comments
Closed

reorganize touch menu items #2562

houqp opened this issue Feb 22, 2017 · 25 comments

Comments

@houqp
Copy link
Member

houqp commented Feb 22, 2017

Sub ticket created for general UX discussion from23 #2555.

@pazos created proposed a solution to replace the home tab with misc items:

test2

I think this is a good step forward and will be happy to accept the change. We need to change the home icon with a new one so people won't get confused and set the wrong expectation if they are already used to the current behavior.

We will probably need better icons for all other tabs too so suggestions welcome.

@retrue
Copy link
Contributor

retrue commented Feb 22, 2017

To replace Home icon I had thought of a door with an arrow. Doing a search on Google Image I have found several ones that I like. Dunno if they are free but something similar could be drawn.

exit1 exit2 exit3

@Frenzie
Copy link
Member

Frenzie commented Feb 22, 2017

Other than pokeball, I don't think the icons are problematic.

  • Gear = settings
  • Pokeball = wtf?
  • Tools = tools
  • Magnifying glass = search
  • Home = home. Should be positioned on the furthest possible right and an off icon of some sort would be clearer because it could be misinterpreted as "KOReader home" as opposed to "OS home". I think putting Exit right underneath Help isn't exactly the most useful. Without an in-depth rewrite I suppose we could stick an extra line in if that's what we wanted to do?

Overall I'd be more interested in adding icons to individual menu entries for quicker visual identification. https://github.com/Templarian has plenty others available. Clock for history, question mark or lifebuoy for help, etc.

@retrue Icons that fit in directly with the current bunch can be found here: https://github.com/Templarian/WindowsIcons/tree/master/WindowsPhone/light

appbar door leave

That guy's a decent source of icons regardless, cf. https://cdn.materialdesignicons.com/light/0.2.63/ (and his blog: http://templarian.com/)

@pazos
Copy link
Member

pazos commented Feb 22, 2017

pokedex icon should be replaced (or deleted) . Right now it has the following items:

on filemanager app: history, open last document, ota update, version and help
on reader app: history, ota update, version and help

help isn't useful at all. Just displays "please report bugs to github.com/koreader". It would be awesome to have some kind of koreader manual on epub, not listed on the filemanager but accesible from the help button.

I think that both OTA update and version are more related to exit (and maybe reboot, poweroff..) than to history or last document.

@Frenzie
Copy link
Member

Frenzie commented Feb 22, 2017

True, so maybe replace pokeball with clock or some such to indicate history. I can't find an appropriate history icon atm so I'll just drop this one (appbar.timer.rewind) 'cause I have to leave right now.

appbar timer rewind

There's also this leaving home icon:
appbar home variant leave

But if it's a traditional help menu I'm not sure if something like that is appropriate. Maybe stick with home after all?

@KenMaltby
Copy link

The original suggestion seems a lot like "About" and could include the reference to the WiKi (like the current help references "Issues".

As I remember it, the Home icon originally brought you back to the File Manager, and to the folder that was "Set as HOME directory", so the logic was sound. Perhaps the Home icon could offer a menu with "Go to Home directory" as its first option. Then an "Exit" icon like one of these: https://duckduckgo.com/?q=exit+icon+png&t=h_&ia=images&iax=1&iai=http%3A%2F%2Ficons.iconarchive.com%2Ficons%2Foxygen-icons.org%2Foxygen%2F256%2FActions-application-exit-icon.png The current "File list" icon could be removed.

The Help option launching an .epub manual (or converted WiKi, perhaps) is a great idea. Maybe with a "Quick Start" section that could be offered in a popup message box (with an option to: "View Quick Start"), to new users. It would include a check box to have it not appear on future start ups.

@Frenzie
Copy link
Member

Frenzie commented Feb 22, 2017

Note that #1371 is about bundling the wiki, although I'm not entirely sure if it's currently sufficiently structured for that. It could be worse though.

@pazos
Copy link
Member

pazos commented Feb 23, 2017

Quick preview of @Frenzie suggested history icon in koreader (with the changes in home menu). The icon hasn't the same size of the others, but you'll get the idea.

On the file manager:
file manager 1
file manager 2

On the reader:
reader 1
reader2

Al alternative order in top menus:
reader-alt

Opinions?

@Frenzie
Copy link
Member

Frenzie commented Feb 23, 2017

I like it except that it's a timer rather than a clock. Unfortunately the license of the current icons is no derived works, so I can't do something obvious like combining appbar.clock with the rewind arrow.

PS This is appbar.clock:
appbar clock

@pazos
Copy link
Member

pazos commented Feb 23, 2017

Another preview, with an icon from wikimedia.

preview

help & version should go in home menu

@pazos
Copy link
Member

pazos commented Feb 23, 2017

Also, in the code, pokedex menu is called the "info" menu. So "Book status" makes sense in this menu, but not on history menu

@Frenzie
Copy link
Member

Frenzie commented Feb 23, 2017

Right, in that case I'd move book status over to the bookmark menu. Anyway, it's just a thought as several people have requested direct access to history.

For info there's appbar.information.circle and appbar.information.
appbar information circle appbar information

That icon from Wikipedia isn't bad although I'd make some adjustments. What's the source?

@pazos
Copy link
Member

pazos commented Feb 23, 2017

Is part of google's material icons.
https://commons.wikimedia.org/wiki/File:Ic_history_48px.svg

@Frenzie
Copy link
Member

Frenzie commented Feb 23, 2017

See, I like that pointy end in the other icon, here's a quick edit. :-P

Ic_history_48px.svg.zip

ic_history_48px svg

@houqp
Copy link
Member Author

houqp commented Feb 24, 2017

OK, so looks like the concensus here is to repurpose home tab as "system tab" which includes the following items:

  • help
  • version (about)
  • update
  • exit

In this case, a home icon is not an accurate representation of the functionalities.

Allocating one tab for time related features seems like a waste because I can't think of other related things that can be put into there. Perhaps we can introduce a generic "action/feature" tab?

@Frenzie
Copy link
Member

Frenzie commented Feb 24, 2017

Speaking only for myself, pokeball → history is by far what I do the most so I'm sympathetic to the various requests for basically a dedicated button. But the icon discussion is largely separate and should probably be a different issue.

My proposal would be to kill the pokeball, move book status over to either page corner bookmark or tools, and stick the rest of pokeball under the "file" menu.

appbar page corner bookmark

  • Table of contents
  • Bookmarks
  • Go to
  • Skim to
  • Follow Links
  • Book status

appbar pokeball

  • Gone

appbar.lines.horizontal (hm, isn't the hamburger menu supposed to be only 3? :-P)
appbar lines horizontal 4

  • History
  • Open last document
  • Update
  • Help
  • Version
  • Exit

Allocating one tab for time related features seems like a waste because I can't think of other related things that can be put into there. Perhaps we can introduce a generic "action/feature" tab?

Not necessarily a bad idea, but how do you distinguish between "tools" and "generic action/feature"? Even without adding icons to the mix the distinction is unclear to me.

One thing that could be done is to separate out finding-related tools. Dictionary, Wikipedia, Fulltext search, Goodreads. This could declutter the tools menu.

@pazos
Copy link
Member

pazos commented Feb 25, 2017

@Frenzie: book status makes sense in bookmarks, all those items are related to the book itself.

I miss a "file manager menu". One that include the items in the search tab and both history and open last document, and maybe some options just for the file browser (show hidden files, sorting..)".

I also feel that the settings tab should display the same items on both file browser and reader app, like network, screen, language, night mode....

@pazos
Copy link
Member

pazos commented Feb 25, 2017

also, replace pokedex on reader app with a search tab with: search in dictionary, search in wikipedia and search full text

@Frenzie
Copy link
Member

Frenzie commented Feb 25, 2017

@pazos

book status makes sense in bookmarks, all those items are related to the book itself.

Maybe, maybe not. They're all navigational and in that sense it'd make more sense in the hamburger menu on the right. The problem in a sense is that status kind of mixes "book info/properties" and "book status" together. As a fun exercise, here's the "book information" (from the file manager) implemented as a reader menu item:

table.insert(self.tab_item_table.info, {

					text = _("Book information"),

					callback = function()
						local FileManager = require("apps/filemanager/filemanager")
						local KeyValuePage = require("ui/widget/keyvaluepage")
						local book_info_metadata = FileManager:bookInformation(self.ui.document.file)
						DEBUG(book_info_metadata)
						if  book_info_metadata then
							UIManager:show(KeyValuePage:new{
								title = _("Book information"),
								kv_pairs = book_info_metadata,
							})
						else
							UIManager:show(InfoMessage:new{
								text = _("Cannot fetch information for a selected book"),
							})
						end
						UIManager:close(self.file_dialog)
					end,
})

Although I'd add that I'd like this:

screenshot_2017-02-25_09-16-33

To go more in this direction (Evince properties dialog):

screenshot_2017-02-25_09-16-17

But that's completely off topic.

I also feel that the settings tab should display the same items on both file browser and reader app, like network, screen, language, night mode....

But it does? Those are the common settings.

also, replace pokedex on reader app with a search tab with: search in dictionary, search in wikipedia and search full text

Yup, already proposed it above! Probably insufficiently clear. ;-)

I'll reorganize the menus today and stick it in a PR with screenshots.

Frenzie added a commit to Frenzie/koreader that referenced this issue Feb 25, 2017
* Replace "home" (exit) menu with hamburger
* Discard pokeball and put items in hamburger
* Introduce search menu in reader

Cf. koreader#2562
This was referenced Feb 25, 2017
Frenzie added a commit to Frenzie/koreader that referenced this issue Feb 25, 2017
* Replace "home" (exit) menu with hamburger
* Discard pokeball and put items in hamburger
* Introduce search menu in reader

Cf. koreader#2562
houqp pushed a commit that referenced this issue Feb 26, 2017
* Replace "home" (exit) menu with hamburger
* Discard pokeball and put items in hamburger
* Introduce search menu in reader

Cf. #2562
@Frenzie
Copy link
Member

Frenzie commented Feb 28, 2017

@Hzj-jie @pazos Taking this out of #2572

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.

With the caveat that I don't think different modes should necessarily have the same hamburger menu considering they're different modes, I agree with the wish to unify the modes as much as possible. I'd like to do so by adding rather than substracting, namely by adding (folder) properties to the filemanager and adding open last document to the reader. However, I really do think that recent files/history and properties/info[1] make sense in hamburger. In many file managers there's "properties" much like in document viewers and editors.

To illustrate, here are a couple of screenshots from Thunar as it happens to be my file manager of choice.

Folder properties, probably most commonly accessed by right-clicking on a folder but in this case by File → Properties.
screenshot_2017-02-28_08-15-01

File properties, accessed in this case by right clicking in the very same file manager but in an audio player something similar might be opened through File → Properties.
screenshot_2017-02-28_08-15-17

[1] NB This actually isn't quite the same as book status, but aligns better with the separate book information dialog.

@Frenzie
Copy link
Member

Frenzie commented Mar 27, 2017

Okay, with #2601 finished any further reorganization of the menus (including, e.g., #2648) has ceased to be a laborious process. The default menu configuration can be found in:

It can be overwritten on a per-user basis by creating files named filemanager_menu_order.lua and reader_menu_order.lua in settings/. You only need to copy the (sub)menu you want to customize.

@retrue
Copy link
Contributor

retrue commented Mar 27, 2017

@Frenzie Could you copy the default filemanager_menu_order.lua and reader_menu_order.lua in the wiki?
Adding too some explanation would be even better.
It seems that the options to choose personalised menu and go back to standard menu have not been added yet.

@AlanSP1
Copy link

AlanSP1 commented Mar 27, 2017

Yes, I think this should be documented, if it isn't already in wiki and any other documentation there is or will be.

@Frenzie
Copy link
Member

Frenzie commented Mar 27, 2017

It seems that the options to choose personalised menu and go back to standard menu have not been added yet.

Right, this issue is unrelated to #2564 except in the sense that easily allowing user customization is a natural result of making menu configuration sensible. This is about where to place menu items, separators, etc. which was previously only partially possible.

@Frenzie
Copy link
Member

Frenzie commented Mar 30, 2017

Copying this where it belongs so it won't get lost.

Typeset menu should be something along these lines:

        -- rendering stuff
        "page_overlap",
        "switch_zoom_mode",
        "set_render_style",
        --sep, I don't think highlight options go with anything else really
        "highlight_options",
        --sep, font-related (typeset) stuff
        "change_font",
        "floating_punctuation",
        "hyphenation",
        --sep, "reading experience" stuff
        "read_timer"
        --speed reading?
  • Move GoodReads over to find menu.

  • Group all of the statistics together with proper separator placement.

  • Rename "Statistics" to "Reading statistics" because we now also have Battery & Storage Statistics

  • Advanced settings & Developer options… tools or hamburger?

Frenzie added a commit to Frenzie/koreader that referenced this issue Apr 6, 2017
* Add separators to navigation menu
* Move speed reading to "typeset" (reading experience) menu
* Move Goodreads over to search

See koreader#2562 (comment)
houqp pushed a commit that referenced this issue Apr 7, 2017
* Add separators to navigation menu
* Move speed reading to "typeset" (reading experience) menu
* Move Goodreads over to search

See #2562 (comment)
@Frenzie
Copy link
Member

Frenzie commented Sep 26, 2017

I'll close this because all of the big stuff is done.

@Frenzie Frenzie closed this as completed Sep 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants