Closed Markus00000 closed 6 years ago
I got the idea for icons that only appear when they can do something useful from the text editor that came with the Cyanogenmod File Manager. Adding a search icon has subsequently complicated the issue, especially for small devices. The rules are
Interesting reasoning. A couple of comments:
I agree the search icon can go on the menu. It is on the toolbar as this appeared to be the convention so that it appears to expand when selected. I don't actually use the app other than for testing updates.
On android 4 the recent apps icon only showed apps that were actually still in memory, I think. Since then it has become another illusion. Apps that were possibly still in memory still appear in recent apps after the device has been shut down, recharged and booted up again. This is obvious with apps like Aldiko, which keep your place in the book you are reading and open back up if still in memory on the same page, but go back to the bookshelf if not, regardless of where it is restarted from.
So if the editor restarts showing the same file in the same position, it was still in memory, if not it wasn't. Also when an app is started by another app is is effectively running in the other app's space, see screen dump.
Editor is running in OI File Manager's space. So it is exceedingly likely that if you restart the editor from the recent apps you can tell what's going on. If you select a file in a file manager and edit it the expected action is to save it and then go back to the file manager, not open another file. That is why the back icon is in the toolbar and the open icon isn't.
Having yet another icon to switch to read only mode when opening a file already achieves that is just the sort of complexity I am trying to avoid. @arschficken made an apposite comment here.
Reading your explanation, it’s reasonable not to show the open icon in that case.
Regarding the edit icon, I think there are two sensible approaches:
A broken feature is worse than a missing one and in this case, it gives a false sense of security. If users might unintentionally exit read-only mode just by rotating their device, I think the feature introduces more risk than not having it in the first place.
My preferences are:
Off-topic: I value your wish to keep Editor simple and greatly appreciate your work. It seems as if your idea of making a simple editor collided with it becoming just too good. As stated in the linked comment, Editor seems to have become the favorite editor of many F-Droid users. While this might bring many feature requests, I hope you manage to keep its complexity at a level that isn’t a burden to you.
This issue can also be closed, right?
I really like being able to toggle edit mode in the new release. Much safer. Sensible icon choice as well.
How you’re handling the save icon now with and without auto-save is excellent, too. It stays in the same place in both cases and allows manual saving even when auto-save is enabled. I find it intuitive and predictable.
OK, I'll close it.
I would prefer if the top bar icons were always the same and in the same place. Using icons becomes easier and faster if their position remains unchanged.
As space is limited in the top bar, the question is which icons should be shown. A complex solution would be to make it configurable: Let users pick which menu items to display in the top bar.
Alternatively, one could try to estimate how often icons are used and determine which icons carry important information. My guesses:
Considering this uncertainty, maybe a menu option that displays a list of possible icons with checkboxes to add them to the top bar is a good solution?