Opened 4 years ago

Closed 4 years ago

#5403 closed defect (fixed)

debloat Blast input and output dialogs

Reported by: Elaine Meng Owned by: Zach Pearson
Priority: high Milestone: 1.3
Component: Sequence Version:
Keywords: Cc: pett
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

From: Eric Pettersen <pett@…>
Subject: Widget spacing
Date: September 14, 2021 at 12:06:18 PM PDT
Hi Zach,

Elaine pointed out to me that the Blast Protein widget layout puts a lot of space between the various widgets, and as you know screen real estate is kind of at a premium in the ChimeraX interface. Anyway, you can pack the widgets more tightly by using .setContentsMargins(0,0,0,0) on your layout, and possibly .setSpacing(0) as well. The tool.py file in the modeller bundle is an example.

--Eric

Attachments (2)

Screen Shot 2021-10-13 at 1.38.38 PM.png (55.8 KB ) - added by Zach Pearson 4 years ago.
spacing 0, margins normal
Screen Shot 2021-10-13 at 1.36.26 PM.png (62.3 KB ) - added by Zach Pearson 4 years ago.
spacing 0, margins 0

Download all attachments as: .zip

Change History (7)

comment:1 by Zach Pearson, 4 years ago

I've pushed a partial fix for the tool -- but should I also reduce the margins on the results panel? I think it looks worse without them.

by Zach Pearson, 4 years ago

spacing 0, margins normal

by Zach Pearson, 4 years ago

spacing 0, margins 0

comment:2 by pett, 4 years ago

I think either one is okay, although I feel it would be nice if the "Query:..." text were centered instead of left justified. I have committed a change to ItemTable that will tighten up the spacing between and around the table-control widgets, so let's look at this again tomorrow and see what we think.

comment:3 by pett, 4 years ago

Cc: pett added

in reply to:  6 comment:4 by Elaine Meng, 4 years ago

Spacing in both input and output dialogs of Blast Protein looks fine to me in the 10-14-2021 daily build of 1.4.  So from my point of view, this could be resolved as fixed if it's also done the same way in the 1.3 branch. Thanks!

comment:5 by Zach Pearson, 4 years ago

Resolution: fixed
Status: assignedclosed

Great! Fixed in this commit.

Note: See TracTickets for help on using tickets.