Project

General

Profile

Bug #10814

List of views in header is making the window too big

Added by Jérémie Projer almost 4 years ago. Updated about 3 years ago.

Status:
Fixed
Priority:
Low
Assignee:
-
Category:
General
Target version:
Start date:
12/25/2015
Due date:
% Done:

100%

Estimated time:
18.88 h
Affected Version:
git development version
System:
all
bitness:
64-bit
hardware architecture:
amd64/x86

Description

In the following modes, I can't maximise the window to the complete screen with the maximise button or with a double clock on the menu bar (i.e, I can't see everthing and I have a big but moving window):
-Lightroom
-Darkroom

It is working in the following modes:
-Map
-Print

I didn't tried it in the tethering mode as I don't use it. Everthing worked fine with Darktable 1.6.9. Attached you can find two screenshots: one of the Print mode (where everything is fine) and one of the Darkroom mode (with the maximum windows size I can have).

Best regards, and thaks a lot for your awesome work.

Darkroom mode.png (644 KB) Darkroom mode.png Jérémie Projer, 12/25/2015 08:01 PM
Print mode.png (428 KB) Print mode.png Jérémie Projer, 12/25/2015 08:01 PM
Bildschirmfoto147.png (4.87 KB) Bildschirmfoto147.png Tobias Ellinghaus, 01/28/2016 12:09 PM
Bildschirmfoto148.png (9.01 KB) Bildschirmfoto148.png Tobias Ellinghaus, 01/28/2016 12:09 PM
Bildschirmfoto149.png (4.17 KB) Bildschirmfoto149.png Tobias Ellinghaus, 01/28/2016 12:09 PM

Associated revisions

Revision 46c65739 (diff)
Added by Tobias Ellinghaus almost 4 years ago

Put most view labels into a dropdown

Only show lighttable and darkroom directly, the rest has to be chosen
from the dropdown to switch views.

TODO:
- make it look a little nicer
- when trying to switch to a view that can't be entered (tethering) it
will keep that entry visible

Fixes #10814

History

#1 Updated by Simon Harhues almost 4 years ago

Just as additional information: Works for me using Ubuntu 15.10 and dt 2.0

If you start dt from a terminal and try it again, do you get any output/error messages in the terminal?

#2 Updated by Jérémie Projer almost 4 years ago

I started Darktable from the terminal and played a bit with it. I had no output nor error messages appering.

#3 Updated by Jérémie Projer almost 4 years ago

Update: I found on the darktable website, under the third release candidate announcement, a person having the same problem as me. One of the asnwer is to change the la guge to english. And effectively, it is working (I can make a screenshot if needed). But it would be nice that it works in the native OS language. For me freench for example.

#4 Updated by Simon Harhues almost 4 years ago

Interesting. As I said for me it works; using German. Strange that it seems to be related to the language...

#5 Updated by Tobias Ellinghaus almost 4 years ago

  • System changed from Ubuntu to all
  • Affected Version changed from 2.0.0 to git development version
  • % Done changed from 0 to 20
  • Target version deleted (Candidate for next patch release)
  • Status changed from New to Triaged
  • Subject changed from Fullscreen mode in Ubuntu 14.04 Trusty to List of views in header is making the window too big

The problem is the ever growing list of views in the top panel. These make the window wider than what you screen can show. If you don't need the info text in the top and know how to switch views without clicking on them you can just hide the bigger part of the header with ctrl-h.
For the future we need to come up with something smaller like a dropdown for secondary views (i.e., everything but lt and dr).

#6 Updated by Jérémie Projer almost 4 years ago

Super, merci beaucoup, thanks a lot. The ctrl -h trick works. Indeed, it is the samll text that tells me how many files in my coolection are selected that takes too much space between the darktable logo and the lightable menu on the top. That's annoying, but I will ust the trick you told me.

For now, as to wait till you come with something new (your idea sounds really great), why not change the translation from "x image sur y de la collection courante est sélectionnées" by "x image sur y sélectionnée"? Way shorter, same meaning. As I know quite well english, if needed I would be happy to help for translations.

#7 Updated by Pascal Obry almost 4 years ago

In fact the translation you're proposing is even better to me. The selected images are not from the current collection but all displayed images. Those images can be part of multiple collections and even from the whole set of images in the library. Let me change this translation which is quite long in French anyway.

#8 Updated by Pascal Obry almost 4 years ago

The text is now as you suggested: "x image sur y sélectionnée" or for plural form "x images sur y sélectionnées".
The meaning is there and the text is way shorter.

As this is just a "workaround" (and only for French language) not sure I should close this ticket.

Tobias what do you think?

#9 Updated by Tobias Ellinghaus almost 4 years ago

Leave it open for now.

Btw, " The selected images are not from the current collection but all displayed images" doesn't make sense. "Collection" and "displayed images" are the same, unless you understand the latter to only include those currently visible due to window size/scrolling.

#10 Updated by Antonio Ospite almost 4 years ago

Hi, this happens with the Italian language too: I am using a 1280x1024 resolution and the window gets too wide and won't resize correctly.

Can you test if the issue present itself also with the English language at smaller resolutions?

Ciao,
Antonio

#11 Updated by Tobias Ellinghaus almost 4 years ago

  • % Done changed from 20 to 100
  • Status changed from Triaged to Fixed

#12 Updated by Jérémie Projer almost 4 years ago

Thanks a lot. I'm impatient to see how it will look like.

#13 Updated by Tobias Ellinghaus almost 4 years ago

Like this. There are still some rough edges, but at least the list doesn't grow any bigger.

#14 Updated by Jérémie Projer almost 4 years ago

Really nice. It looks really good like this.

#15 Updated by Roman Lebedev about 3 years ago

  • Target version set to 2.2.0

Also available in: Atom PDF

Go to top