Skip to content

VPN-7711 Fix layout issues on ContactUs form with some locales - #11444

Merged
encrypt94 merged 3 commits into
mainfrom
VPN-7711-contact-support-ui-issues-some-locales
Jul 20, 2026
Merged

VPN-7711 Fix layout issues on ContactUs form with some locales#11444
encrypt94 merged 3 commits into
mainfrom
VPN-7711-contact-support-ui-issues-some-locales

Conversation

@encrypt94

@encrypt94 encrypt94 commented Jul 17, 2026 •
edited by atlassian Bot
Loading

Copy link
Copy Markdown
Collaborator

Description

The contact form overflowed to the right for some locales.

The issue was introduced by me in the share-logs PR, which removed the Column { Layout.fillWidth: true } wrapper that used to bound MZLinkButton and left it as a direct child of the ColumnLayout still using:

anchors.left: parent.left
anchors.right: parent.right

Fix a couple of warnings too.

Reference

VPN-7711

Checklist

  • My code follows the style guidelines for this project
  • If bug/feature is at all notable, I've added a draft release note as a comment in addons/strings.yaml.
  • I have not added any packages that contain high risk or unknown licenses (GPL, LGPL, MPL, etc. consult with DevOps if in question)
  • I have performed a self review of my own code
  • I have commented my code PARTICULARLY in hard to understand areas
  • I have added thorough tests where needed

@encrypt94
encrypt94 requested review from artfwo and oskirby July 17, 2026 16:48
@encrypt94
encrypt94 merged commit b3b22b7 into main Jul 20, 2026
140 of 141 checks passed
@encrypt94
encrypt94 deleted the VPN-7711-contact-support-ui-issues-some-locales branch July 20, 2026 08:05
@encrypt94 encrypt94 mentioned this pull request Jul 20, 2026
Merged
6 tasks
encrypt94 added a commit that referenced this pull request Jul 20, 2026
@encrypt94
VPN-7711 Fix layout issues on ContactUs form with some locales (#11444)…
1a42449 
… (#11445)

* replace anchors with Layout.fillWidth true

* fix warning Unable to assign [undefined] to double

* fix warning  Duplicate signal name: invalid override of property change signal or superclass
mcleinman pushed a commit that referenced this pull request Jul 20, 2026
@encrypt94 @mcleinman
VPN-7711 Fix layout issues on ContactUs form with some locales (#11444)…
79b7a88 
… (#11445)

* replace anchors with Layout.fillWidth true

* fix warning Unable to assign [undefined] to double

* fix warning  Duplicate signal name: invalid override of property change signal or superclass
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@artfwo artfwo artfwo approved these changes
@oskirby oskirby Awaiting requested review from oskirby

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@encrypt94 @artfwo