Skip to content

Conversation

@glughi
Copy link
Contributor

@glughi glughi commented Nov 17, 2025

What this PR does

Fixes #7726

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

@zoran995
Copy link
Collaborator

Hi @glughi thanks for finding this. Can we completely remove the fixed width from this button and use something as

  • url input wrapper - flex layout
  • url input text box - height auto, width: 100%
  • url input add button - width unset; max-width: 150px; word-wrap: break-word;

This way, any text length should work

Copy link
Collaborator

@zoran995 zoran995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Collaborator

@niktayv niktayv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested it, and everything worked well.
The code changes LGTM, save for a couple of nitpicks.

@niktayv niktayv merged commit 2256d81 into TerriaJS:main Jan 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add data from web button too short

3 participants