Opened 5 years ago

Last modified 5 years ago

#3681 assigned defect

Molecule Display toolbar icons in different orders, sometimes nucleotides before others

Reported by: Elaine Meng Owned by: Greg Couch
Priority: moderate Milestone:
Component: UI Version:
Keywords: Cc: pett
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

See attached screenshots showing the Molecule Display tool with ChimeraX version numbers shown in Log. In the daily build screenshot the nuc-style icon is to the left of the stick/sphere/ball icons, and the nuc-color icon is to the left of the other coloring icons. In the release candidate screenshot they are after the others within the respective sections (style, coloring).

Attachments (2)

dailybuild.png (541.7 KB ) - added by Elaine Meng 5 years ago.
releasecandidate.png (543.0 KB ) - added by Elaine Meng 5 years ago.

Download all attachments as: .zip

Change History (9)

by Elaine Meng, 5 years ago

Attachment: dailybuild.png added

by Elaine Meng, 5 years ago

Attachment: releasecandidate.png added

comment:1 by Elaine Meng, 5 years ago

earlier in the day I could have sworn it was the other way around, although the two versions were still different, so maybe it is a semirandom thing depending on when the nucleotides bundle is started

comment:2 by Greg Couch, 5 years ago

The ordering must be under specified. I'll look into it.

comment:3 by Greg Couch, 5 years ago

Cc: pett added

Eric, will the recent provider changes, is there any chance that the providers could be added in a different order than how they appear in the bundle.xml file?

comment:4 by Greg Couch, 5 years ago

will -> with

comment:5 by pett, 5 years ago

There is no specific order that add_provider is called for providers within a bundle, and there's _really_ no order for providers coming from different bundles. Typically when I need providers ordered, either the manager imposes an order (e.g. alphabetization) or the providers include ordering information in their Provider tags (e.g. preset providers have an order="number" tag attribute) which is passed into the add_provider call. The manager then typically puts the providers in order during the end_providers call.

It occurs to me that the toolbar manager could be converted into a lazy manager, but that would only be a "win" in the 0.0001% case where the user has turned off autostart of the Toolbar tool -- so really not worth the effort.

comment:6 by Greg Couch, 5 years ago

I was depending on the previous behavior where the providers were registered in the order that they were given in the bundle.xml file. Will add constraints.

comment:7 by pett, 5 years ago

AFAIK, I didn't change anything in that regard. On the other hand, it was just side-effect behavior, so you shouldn't depend on it for sure.

Note: See TracTickets for help on using tickets.