Opened 3 years ago

Closed 3 years ago

#329 closed defect (fixed)

Button and checkbox alignment in FOC could be improved

Reported by: lewisr Owned by:
Priority: trivial Milestone: 1.4.0
Component: UI Version: 1.4.0
Keywords: refresh, update, view, monitor Cc: ataylor

Description

Related to the changes made in #318, the new checkbox and Help button positioning is still not quite right.

Attaching images of current and proposed positioning.

Attachments (9)

FOC_button_alignment.png (2.3 KB) - added by lewisr 3 years ago.
Current button and checkbox alignment, after changes made in #318
FOC_button_alignment_2.png (1.7 KB) - added by lewisr 3 years ago.
Proposed alignment of buttons and checkbox
FOC_button_alignment_small_96dpi.png (7.0 KB) - added by lewisr 3 years ago.
SNAP with SDDFONTSIZE small and SDDFONTDPI 96
FOC_button_alignment_small_120dpi.png (7.3 KB) - added by lewisr 3 years ago.
SNAP with SDDFONTSIZE small and SDDFONTDPI 120
FOC_button_alignment_medium_96dpi.png (8.3 KB) - added by lewisr 3 years ago.
SNAP with SDDFONTSIZE medium and SDDFONTDPI 96
FOC_button_alignment_medium_120dpi.png (8.7 KB) - added by lewisr 3 years ago.
SNAP with SDDFONTSIZE medium and SDDFONTDPI 120
FOC_button_alignment_large_96dpi.png (8.9 KB) - added by lewisr 3 years ago.
SNAP with SDDFONTSIZE large and SDDFONTDPI 96
FOC_button_alignment_large_120dpi.png (9.3 KB) - added by lewisr 3 years ago.
SNAP with SDDFONTSIZE large and SDDFONTDPI 120
open dialog help button.png (17.1 KB) - added by gyoung 3 years ago.

Download all attachments as: .zip

Change History (20)

Changed 3 years ago by lewisr

Current button and checkbox alignment, after changes made in #318

Changed 3 years ago by lewisr

Proposed alignment of buttons and checkbox

comment:1 Changed 3 years ago by gyoung

This is going to be a problem. On my system while the checkbox is about where you show it on the current screen capture my help button is already where you show it on the proposed alignement. I can't move it any further right without taking it off the dialog.

comment:2 Changed 3 years ago by abwillis

Interesting, I just pulled the latest svn and built and my Help button only shows He as the rest of the button is cut off on the right side.

comment:3 Changed 3 years ago by lewisr

I've asked Keith to weigh in on this. I've picked through the PM references I have and even perused some of the PM articles in EDM/2. I'm not finding it, though. My thinking is that we can create a container on top of the file dialog, properly aligned, and place the button in that. That doesn't help with the checkbox, of course.

I suspect this is because my screen res is 1400x1050 and my SNAP settings are:

SET SDDFONTDPI=96

comment:4 Changed 3 years ago by lewisr

If we can't get this perfectly right-aligned for all cases, we should at least move it to the left so that we're not clipping the button.

Can we get this change done in the near future so we can get to GA?

Changed 3 years ago by lewisr

SNAP with SDDFONTSIZE small and SDDFONTDPI 96

Changed 3 years ago by lewisr

SNAP with SDDFONTSIZE small and SDDFONTDPI 120

Changed 3 years ago by lewisr

SNAP with SDDFONTSIZE medium and SDDFONTDPI 96

Changed 3 years ago by lewisr

SNAP with SDDFONTSIZE medium and SDDFONTDPI 120

Changed 3 years ago by lewisr

SNAP with SDDFONTSIZE large and SDDFONTDPI 96

Changed 3 years ago by lewisr

SNAP with SDDFONTSIZE large and SDDFONTDPI 120

comment:5 Changed 3 years ago by lewisr

Almost there. Of my recent tests, it appears that only with SDDFONTSIZE=large is the Help button being positioned too far to the right. In a perfect world, this would align nicely to the right, with the same padding as we have for the OK button on the left, but for now, a little more pull to the left, and we should be fine for release of 1.4.0.

FYI, these recent shots were taken with 1.4.0 RC4 using:

12-17-16 16:51 1,408,130 124 Lucide1.dll

(The bldlevel string, however, still reads: Date/Time: 2016-09-11 03:24:00.)

comment:6 Changed 3 years ago by gyoung

I still haven't automated updating the build level string. I will try to do that in the next few days but failing that I will just update it manually for the release and worry about automating it later. I am going to try right aligning the Help button with files listbox and see how that works.

Changed 3 years ago by gyoung

comment:7 Changed 3 years ago by gyoung

How about this? Maybe it will even stay put on all systems. I noticed the make thumbnail checkbox stayed where it was placed on all of these. My png is panorama at what ever the default is.

comment:8 Changed 3 years ago by lewisr

I could live with that. Please send me the dll and I'll give it a go here with SNAP at a few resolutions.

comment:9 Changed 3 years ago by lewisr

  • Cc ataylor added
  • Keywords refresh update view monitor added

Yes, this latest one works fine, even with SNAP set for SDDFONTSIZE large and SDDFONTDPI 120. My only criticism about the FOC would be that at that scaling, the right border of the preview pane is a little close to the right edge of the FOC, but that's a very, very minor nit, and one we might have just as hard a time fixing as the positioning of the Help button, so I'd say leave it (note the spacing in FOC_button_alignment_large_120dpi.png).

Thanks, Gregg!

comment:10 Changed 3 years ago by gyoung

Don't know what to tell you. There is something about the preview pane that screws up the dialog spacing. I initially figured I had some math error in the spacing. I went in and check first oddity was the preview pane was only set at 24 units wide. It is clearly 120 or more. Also originally the help button should have been partially off the dialog on all screens as it extended beyond the width of the dialog. I fixed these but it did really change anything. I just tried making the dialog 100 units wider that also didn't change anything (same narrow boarder on the right). I have no idea why it behaves this way. Probably not worth worrying about.

comment:11 Changed 3 years ago by lewisr

  • Resolution set to fixed
  • Status changed from new to closed

Agreed 100%.

Closing this as fixed.

Thanks!

Note: See TracTickets for help on using tickets.