hius07
hius07
Swipe north deletes a word together with a space.
> swipe north still gets me stuck with that: Yes, it's a word delimiter that belongs to the previous word, we go from right to left and stop on it....
For example, in "one,two" if cursor stands between coma and t (there is no space between, bad formatting) - swipe north deletes "two", that is good.
To fix the last issue, we can check if the word is empty, and find the next (to the left) word to delete.
Can you try if it is the desired behaviour. Please note that normally we do not delete the punctuation mark together with a word. So the fix deletes **the punctuation...
Look at it please: you can move https://github.com/koreader/koreader/blob/f894330f1eca81097f9f4c5a5d5c8fd5fb945dea/plugins/coverbrowser.koplugin/main.lua#L296 to the top of `addToMainMenu`, then you will have `fc.display_mode_type` that is false for classic mode, or "list" or "mosaic". You can...
First impressions on wordings and placement: (1) I am not sure that our file browser is a "library". (Collections are closer to "library" or "bookshelf") "Items" are changed to "books",...
We will not be able to have, say, Collections in Mosaic with covers and History in Mosaic without covers, right?
CircleCI warnings: ``` plugins/coverbrowser.koplugin/main.lua:96:66: line contains trailing whitespace plugins/coverbrowser.koplugin/main.lua:102:21: line contains trailing whitespace plugins/coverbrowser.koplugin/main.lua:151:16: line contains trailing whitespace plugins/coverbrowser.koplugin/main.lua:471:50: line contains trailing whitespace ```
> another good idea Keep the menu structure as is, change the wordings only.