Project

General

Profile

Feature #8820

Screen size detection for viewport default settings

Added by Pascal de Bruijn almost 6 years ago. Updated almost 5 years ago.

Status:
Triaged
Priority:
Medium
Assignee:
-
Category:
Darkroom
Target version:
Start date:
07/08/2012
Due date:
% Done:

20%

Affected Version:
System:
bitness:
64-bit
hardware architecture:
amd64/x86

Description

I was taking a look at our dutch translations (courtesy of Ger), so I created a new clean user account with the dutch locale. When I started Darktable and switched to Darkroom mode, I noticed the viewport is too small (eventhough it's set to 1300x1000 in preferences. When I changed those values to 1280x800 (the size of my screen) and restarted Darktable everything was fine.

So I'm guessing something with our handling of default preferences isn't entirely alright.

I tested this with our darktable-1.0.x branch (1.0.4+...).

dtviewportfail.png (333 KB) Pascal de Bruijn, 07/08/2012 08:12 PM

History

#1 Updated by Tobias Ellinghaus almost 6 years ago

  • % Done changed from 0 to 20
  • Priority changed from High to Medium
  • Status changed from New to Triaged

This is done on purpose when "slow" hardware is detected. See 0d743965a6a71f9ff8dbcd1f0e7f661e10e08d4a. However I agree that this is not ideal, so I'll not close this ticket.

#2 Updated by Pascal de Bruijn almost 6 years ago

Aha...

The funny thing is, on second start, it does set it to 1300x1000, so that procedure still concludes my system is fast enough.

I guess we should probably display a message about this behavior, so the user is aware that on first start darktable's behavior might be different.

Also setting this value to 1300x1000 makes little sense on 1280x800/1366x768 sized displays. Do we already have screen-size detection in there, so we might be a bit more intelligent about this?

#3 Updated by Simon Spannagel over 5 years ago

  • Subject changed from Darktable/Darkroom mode viewport to small to Screen size detection for viewport default settings
  • Tracker changed from Bug to Feature

Updated the description.

We don't have screen resolution detection currently. But since this bothers quite some first-time users it would be nice to have.

#4 Updated by Simon Spannagel almost 5 years ago

  • Target version changed from Candidate for next patch release to Future

not a bugfix.

Also available in: Atom PDF