GoAlexander / harbour-editor

Functional rich editor for Sailfish OS.
GNU General Public License v3.0
17 stars 16 forks source link

[Usability issue] Eliminate large empty vertical spaces in Editor.'s pulley menu #81

Closed Olf0 closed 6 years ago

Olf0 commented 7 years ago

Please eliminate the large empty vertical spaces in Editor.'s top pulley menu:

This usability issue is much more prominent in landscape orientation:

screenshot_jolla1-landscape

screenshot_jolla1-portrait

GoAlexander commented 7 years ago

First of all I want to say that "UI 2.0" is in hard development. I will change some logic. By this time I replaced position of toolbar and menu: photo_2017-05-27_15-52-03

The next step is changing of pulley menu - it will be (I think) without the buttons it will contain only MenuItems (as Settings option) but the will be special external page where will be all NOT often using buttons. This changing is on the basis of all user`s feedback and my personal experience :)

The main principles of new design: provide more "easy-to-use experience" + to comply with the principles of Sailfish OS system`s design.

GoAlexander commented 7 years ago

Eliminate the large space above "Settings".

I can`t change it because it is default behavior of the system. (I saw something like you said in apps of CodeRus but it is something like hacking I think :) )

Shrink the vertical margins around the file path, if possible (its font size is good and should stay, IMO). You might even eliminate the large empty space below the pulley menu's second icon row (i.e. above the regular top screen icon row) completely, when no file path is currently set, but that may not be worth the effort.

I know this bug and I don`t know how to fix it :(

The vertical space between the two icon rows in the pulley menu seems to be a bit larger than the vertical space between the word "Settings" and the first icon row there: If I am right, please shrink the former vertical space to the size of the latter.

In case of UI 2.0 I think it will be no longer relevant.

[Further suggestion] The icons in the regular (top screen) icon row below are (rightly) all "grayed out", when the pulley menu is pulled. Hence it would be a better use of the available screen space not to display them at all in this situation.

Good suggestion. I will mention it in my ToDo but will implement it only in far future ) (there is another priorities)

[Even further reaching suggestion] Separate out the "Settings" pulley entry to a bottom pulley.

Also not longer relevant in case of UI 2.0 but I will explain one think that I see not all users of Editor understand: I implemented toolbar also for the case when you are in the center of large document and do not want to scroll at the top to open menu. With toolbar you can press it and swype and menu wiil be opened in the center of the document!

So if I will place Settings in another menu I have to provide additional toolbar which will reduce space for text...

Olf0 commented 7 years ago

Hello @GoAlexander, a. The screenshot of Editor.'s UI 2.0 with the moved toolbar (to the top of the vKeyboard) looks like a good / improved layout. b. Unfixable bugs: Well, that's life. ;) And for the one, which may be fixable, you already have issue #61. c. As you noted the only left over item (not covered by a. or b.: to "gray out" the toolbar, when it is inaccessible, anyway), closing this issue.

Olf0 commented 7 years ago

Well, the situation is improved a bit with the "UI 2.0", which first appeared in Editor. v0.8:

But ultimately the bullet points # 1 ("unfixable"), # 3, # 4 and # 5 in my original posting (with which this issue was opened) still hold true, currently. :\

See screenshot_editor -0 8_bottom-pulley and screenshot_editor -0 8_bottom-pulley-above-vkeyboard

Tested with Editor. v0.8 on a Jolla 1 phone under SailfishOS 2.1.0.11

GoAlexander commented 6 years ago

@Olf0 Do you like it? I reduced the space. :) https://www.dropbox.com/s/6lhsqerzieex1mg/Screenshot_20180424_001.png?dl=0

I can to reduce it more but I think that it will be not comfortable to choose this item in this case.

Olf0 commented 6 years ago

The vertical spaces above and below that pulley menu entry "Settings" are fine with Editor. 0.8.4.
And it makes a difference, especially in landscape orientation:

screenshot_20180425_002

Side note: I wonder, why the vertical space above the file path looks quite large in your screenshot, but not on my Jolla 1 with Editor. 0.8.4 (regardless if using it in landscape or portrait orientation). Well, maybe this is only my visual perception.

Thanks!

P.S.: Really sad, that the font size of the file path is not adjustable for you AFAIU, or is it only its vertical margins? Alternatively to my original suggestion of shrinking its margins, if making the file path's font size a bit larger is possible (at most the same font size as the word "Settings"), the whole pulley menu would look much more in proportion. If the text line (containing the file name) becomes too long, it would have to be made right aligned (so the file name is always visible) or scrolling horizontally; or always simply display the file name proper (without the rest of the path), which is less likely (although still able) to be too long. Never mind, if this poses significant hurdles (I suppose so): The pulley menu layout is O.K. in its current state (as of v0.8.4) and there are definitely more important issues and possible enhancements to care about.

Olf0 commented 6 years ago

BTW & OT: Editor. 0.8.3 was not available in the Jolla Store (currently v0.8.2 is the latest version there, AFAICS), hopefully v0.8.4 will.

GoAlexander commented 6 years ago

0.8.3 was not able because of a bug in history. 0.8.4 is in Jolla moderation.

GoAlexander commented 6 years ago

I see that I fixed huge part of your requests here ao I will close this issue.