Skip to content

DoubleFloatingButton: Fix appearance, behavior and RTL layout

ivan tkachenko requested to merge work/ratijas/double-floating-stack into master

Fix overlapping separator color

Instead of painting only a single separator with tons of custom logic for tracking previous/next button state like a real Segmented Button would do, just use z index trick. It works fine for a straight 1px line and no shadows (there's only one global shadow for the whole control).

Fix Tab order

Declare left button higher than the right one in file, so it gets prioritized for Tab order.

Sync button radius with top-level control radius

Well-spotted by Mathis Brüchert in !159 (closed)

Drop hard requirement for action properties

Marking properties as required is a bit hostile and not in the spirit of QQC2.

Rename action properties to be RTL-friendly

Old properties are kept around for backward-compatibility reasons.

Implement RTL support

Before this patch, the component looked completely broken in RTL layout.

Fix snippet in doc comment

  • There was a typo in import name;
  • Icon "list-minus" does not exist;
  • Who would put a "Zoom In" button right before "Zoom Out"? They usually go the other way around: just think of a zoom slider in between.

Merge request reports