Skip to content

File browser reshuffle#11781

Closed
Commodore64user wants to merge 0 commit into
koreader:masterfrom
Commodore64user:file-browser
Closed

File browser reshuffle#11781
Commodore64user wants to merge 0 commit into
koreader:masterfrom
Commodore64user:file-browser

Conversation

@Commodore64user

@Commodore64user Commodore64user commented May 8, 2024

Copy link
Copy Markdown
Member

From the makers of the wildly successful Sleep screen rework #11549 and the yet to be merged status bar makeover #11678, We present to you the 'new kid in town', the new file browser (file cabinet) menu.

There has been a far less aggressive approach here, most of the work pertains the reduction of "view modes" and rewording of settings. Some logic was added for the functioning of the simpler view modes but besides that no changes in functionality were added or removed.

I am conflicted about the use of the word "mosaic" here, as what we get is a standard grid view that has nothing to do with actual mosaics, I would love to hear what you think, should it go, should it stay?

the following screenshots were captured on a non-touch kindle 4, so some options might not appear there. Anyway without any further ado,

Here it is the menu upon opening:

after going to the Library view:

Inside mosaic and list view properties:

and finally, inside the file management submenu:

Important

I would like to have the classic view properties greyed out when 'classic mode' NOT selected but the usual enable function does not work in that case, any help there would be greatly appreciated.

Last but not least, thanks to @Frenzie for his help, patience and support.


This change is Reviewable

Comment thread frontend/apps/filemanager/filemanagermenu.lua Outdated
Comment thread plugins/coverbrowser.koplugin/main.lua Outdated
return not BookInfoManager:getSetting("unified_display_mode")
return filemanager_display_mode == "list_image_meta"
or filemanager_display_mode == "list_image_filename"
or filemanager_display_mode == "list_only_meta"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe you can use something like

menu.display_mode_type = display_mode and display_mode:gsub("_.*", "") -- "mosaic" or "list"

to shorten this and similar checks.

Comment thread frontend/apps/filemanager/filemanagermenu.lua
Comment thread frontend/apps/filemanager/filemanagermenu.lua
@poire-z

poire-z commented May 9, 2024

Copy link
Copy Markdown
Contributor

This PR conflicts and does not apply on a current nightly.
image

It's been a while since you started your work, and you haven't updated it for the recent changes to coverbrowser.main.lua, the top 2 commits in:
https://github.com/koreader/koreader/commits/master/plugins/coverbrowser.koplugin/main.lua

Dunno how this is solvable via GitHub web (dunno if you're using that or a real git command line), @hius07 may know what's the best way to resolve it.
Had to manually edit that file on my device, starting from yours, and re-applying these last 2 commits changes (2 "favorites" <> "collections, one local fc = ... to add, 6 to remove, 11 self:refreshFileManagerInstance() to replace with fc:updateItems(1, true)), not sure I got it all right, but if you resolve them manually, here's my final file if you need to compare: fixed_manually_main.lua.txt

@Commodore64user

Copy link
Copy Markdown
Member Author

okay @poire-z I believe I resolved all the conflicts...?

@benoit-pierre

Copy link
Copy Markdown
Member

No, you need to rebase, you're still 47 commits behind master.

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.

4 participants