fix(playground): ui parity bugs 2 - #69
Open
sundram-bruno wants to merge 6 commits into
Open
sundram-bruno wants to merge 6 commits into
sundram-bruno wants to merge 6 commits into
Conversation
sundram-bruno
changed the title
sundram-bruno
marked this pull request as draft
September 7, 2026 11:59
sundram-bruno
force-pushed
the
fix/script-error-cards-layout
branch
from
September 16, 2026 05:15
bc587e5 to
1a01276
Compare
|
…response content scrollable below them
Tables cap at 24rem and scroll inside the container; cell hover tooltips and name title attributes are gone. Multiline values grow to fit instead of scrolling, re-fit when a column is dragged narrower, and share one line-height with single-line inputs so rows keep their height. The header-name suggestions list gets the thin themed scrollbar. Assertions gain the Description column with round-trip persistence and the app's short column labels. The first column header starts exactly where the cell text starts, and query params say Name instead of Key. KeyValueTable styles move into an Emotion StyledWrapper like every other component; the legacy .text-input rules are dropped because HighlightedInput owns those fields.
Switching the dock placement swaps the dock component and remounts the request and response panes, so their locally held tab reset to the default. Both panes now keep the active tab in session storage, the same lane the collapsible sections and dock sizes use, so it survives the remount and a reload.
sundram-bruno
force-pushed
the
fix/script-error-cards-layout
branch
from
September 22, 2026 04:52
1a01276 to
c863479
Compare
sundram-bruno
marked this pull request as ready for review
September 22, 2026 04:54
sundram-bruno
force-pushed
the
fix/script-error-cards-layout
branch
from
September 22, 2026 05:12
1c8606d to
5e1199f
Compare
Parity item 17 was worded ambiguously and the implementation followed the wrong reading, so the table height cap and the removal of the cell tooltips both come out. Tables grow with their content again and a clipped cell shows its full text on hover and through its title.
sachin-bruno
approved these changes
Sep 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Jira: BRU-4444
Description
Parity fixes for the collection docs playground, covering items 18, 21, 24, 26, 29 and 31 of the ticket.
What changed
18. Environment value fields. Multiline value fields grow to fit their content instead of scrolling inside a fixed box, and they re-fit when a column is dragged narrower. Table, card and secret variants now behave the same.
21. Header name suggestions. The suggestions list renders in a portal, so the playground's scrollbar styling never reached it. It now uses the same thin themed scrollbar as the rest of the app.
24. Request tabs across a dock change. Changing the dock placement swaps the dock component and remounts the request and response panes, which reset their tab to the default. Both panes now keep the selected tab in session storage, the lane the collapsible sections and dock sizes already use.
26. Assertion descriptions. The Assertions tab was the only tab without a Description column, although the format and the desktop app both carry the field. It now shows and persists the description, and its column labels match the app (Expr, Value).
29. First column alignment. The first column header now starts exactly where the cell text below it starts, with and without the enable checkbox. The query params table labels that column Name instead of Key, as the app does.
31. Script error cards. When both the post-response and tests scripts failed, closing one error card closed both, and the cards took the panel's height from the content below them. Each card now closes on its own and the response body and test results keep their full height.
Also in this PR
KeyValueTable.cssbecomes an EmotionStyledWrapper, matching every other component in the package. The legacy.text-inputrules are dropped becauseHighlightedInputalready owns those fields.