window is unnecessary large

Bug #1622227 reported by doniks
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
uNav
Fix Released
Low
costales

Bug Description

when switching to desktop mode on the nexus 7 the unav window is quite large. even higher than the screen in landscape and almost as wide as the screen in portrait. i don't think this is needed. a significantly smaller minimum window size would help convergence.

Related branches

Revision history for this message
costales (costales) wrote : Re: [Bug 1622227] [NEW] window is unnecessary large

Thanks for the feedback!
I think the width of nexus 7 is not enough for a convergence layout.
Probable fix: set max width to device width.
A hug.

Changed in unav:
status: New → Triaged
Revision history for this message
doniks (kaabud-lp) wrote :

Thanks for looking into it. It's a small screen indeed.

Personally I haven't made up my mind yet whether I have real use cases for a window layout on the Nexus 7 or whether it's just a gimmick.

In case anybody cares about it, my workaround for the time being is to delete these two lines

minimumWidth: units.gu(60)
minimumHeight: units.gu(80)

from

/custom/click/navigator.costales/current/qml/Mail.qml

This allows me to resize uNav arbitrarily.

Revision history for this message
costales (costales) wrote :

Hi,
Could you try a .click?
As I don't have that device, I only can send you tests.
Thanks in advance.

Changed in unav:
milestone: none → 0.64
assignee: nobody → costales (costales)
status: Triaged → In Progress
importance: Undecided → Low
Revision history for this message
costales (costales) wrote :

Or more easy for you if you want to edit the file:
In Main.qml change these lines:

http://bazaar.launchpad.net/~costales/unav/0.64-window-size-never-as-screen/revision/135

Thanks in advance!

Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :

I could test a click, but indeed if it's just two lines, editing is easier for me.

I have tried by adding the two "max lines" (and restoring the two "min lines" I had commented out). I don't think this is what you want to do. See below my testing steps with numbered screenshots.

1. switch to window mode and make the window as small as I can.

2. Turn into landscape mode - now approximately the bottom third of the window is outside of the screen

3. make the window as large as I can. It is now not possible anymore to stretch to the full width of the screen in landscape. It seems that Screen.width relates to the "native" orientation of portrait. NB: See in the background, how I made the Apps scope almost twice as wide as the landscape screen - it seems that no maximum is imposed there

4. turn into landscape. now the bottom little bit of the window can't be shown on the screen because I cannot move the window above the indicator bar.

then I have taken out the "max lines" again and put back
minimumWidth: units.gu(40)
minimumHeight: units.gu(40)

5. that's how 40*40 looks like on the nexus 7 (7 inch screen), the window is approximately four fingertips wide. If I make is smaller some of the screen controls (zoom buttons, settings button) start to fall off the window. One /could/ still make it a little less high, but below, say 40*30 the scrolling in the settings window becomes awkward.

So, my vote would be min=40*40, no maximum.

Revision history for this message
costales (costales) wrote : Re: [Bug 1622227] Re: window is unnecessary large

Thanks for the testing.

How about this?
http://paste.ubuntu.com/23584739/
could you provide screenshots of that?

I think 40x40 is so small for the search dialog.

--
Sent using Dekko from my Ubuntu device

Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :
Revision history for this message
doniks (kaabud-lp) wrote :

6 and 7 attached.

Revision history for this message
costales (costales) wrote :

It looks fine with 60x60.
How do you see it?

On Dec 5, 2016 10:05 PM, "krusty" <email address hidden> wrote:

> 6 and 7 attached.
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1622227
>
> Title:
> window is unnecessary large
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/unav/+bug/1622227/+subscriptions
>

Revision history for this message
costales (costales) wrote :

@krusty: Hi :) Do you see it OK with 60x60? :) Thanks in advance!

Revision history for this message
doniks (kaabud-lp) wrote :

Well, as I said, my vote would be for 40x40.

I don't want to put words into your mouth, but maybe you see it like: Use the smallest size where you as a developer still feel motivated to provide a *nice* user experience.

Whereas my point of view would be: Use the smallest size, below which users would risk running into really weird errors and they might start bugging developers with useless bugreports, like: "I made the window 1x1 and now I can't make it large again."

It's not like the user would be forced to use the smallest size. The smallest size should just be a kind of safety net, but if a user would want to choose to sacrifice some *nicety* for a particular usecase (whateverthatmightbe), why not let them.

Revision history for this message
doniks (kaabud-lp) wrote :

That being said, there is nothing /wrong/ with 60x60, it just wouldn't be my preference.

costales (costales)
Changed in unav:
status: In Progress → Fix Committed
costales (costales)
Changed in unav:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.