Skip to content

BookShortcuts: open file with associated provider#13106

Merged
hius07 merged 3 commits into
koreader:masterfrom
hius07:boook-shortcut
Jan 25, 2025
Merged

BookShortcuts: open file with associated provider#13106
hius07 merged 3 commits into
koreader:masterfrom
hius07:boook-shortcut

Conversation

@hius07
Copy link
Copy Markdown
Member

@hius07 hius07 commented Jan 20, 2025

For example, viewing crash.log in TextViewer can be assigned to a gesture or a profile.

Also menu prettify.

1

2


This change is Reviewable

@hius07 hius07 added the Plugin label Jan 20, 2025
@Frenzie Frenzie added this to the 2025.01 milestone Jan 20, 2025
Comment thread plugins/bookshortcuts.koplugin/main.lua Outdated
Comment thread plugins/bookshortcuts.koplugin/main.lua Outdated
end
local mode = lfs.attributes(k, "mode")
if mode then
local title = T(_("Open %1"), mode == "file" and k:gsub(".*/", "") or k)
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.

Might be worth a quick translator's comment that it's a filename.

@Commodore64user
Copy link
Copy Markdown
Member

Nice touch, those file type icons. 👍

@mergen3107
Copy link
Copy Markdown
Contributor

Why does the second item have an icon of a trash bin?

Sorry, I am not quite dolphin the development of shortcuts

@hius07
Copy link
Copy Markdown
Member Author

hius07 commented Jan 21, 2025

Why does the second item have an icon of a trash bin?

Deleted file.

hius07 and others added 2 commits January 21, 2025 06:24
Co-authored-by: Frans de Jonge <fransdejonge@gmail.com>
@hius07 hius07 merged commit cda2dca into koreader:master Jan 25, 2025
@hius07 hius07 deleted the boook-shortcut branch January 25, 2025 10:26
Commodore64user pushed a commit to Commodore64user/KOReader_fork that referenced this pull request Jan 27, 2025
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants