Skip to content

InlineMessage: Downstream and clarify padding expressions that are based on background's border width

ivan tkachenko requested to merge work/ratijas/inline-message-padding into master

InlineMessage: Downstream and clarify padding expressions that are based on background's border width

Template implementation assumed too much that background item must be a Rectangle (is it pretty much the only base type that has border.width property). Not only that, but relying on border.width didn't make any sense, since border is drawn by other means (multiple Rectangles with offsets) and it just happened to match the default width value 1 of an invisible border.

Ideally, I'd also reset verticalPadding and horizontalPadding, but those were not available in QQC 2.0, which we can not bump without breaking compatibility with existing apps until KF6 (what a "wonderful" versioning mechanism)

Merge request reports