Skip to content

fix(issue-views): Render editable title in topbar slot for page-frame layout#113488

Merged
JonasBa merged 2 commits into
masterfrom
jb/issues/editable-title-de-1132
Apr 21, 2026
Merged

fix(issue-views): Render editable title in topbar slot for page-frame layout#113488
JonasBa merged 2 commits into
masterfrom
jb/issues/editable-title-de-1132

Conversation

@JonasBa

@JonasBa JonasBa commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Update editable title in page frame mode to render into the topbar slot when editing and reduce the size of the header to use the default size

Fixes DE-1132

… layout

When clicking to edit an issue view title with the page-frame feature
enabled, the editing input rendered into the page body instead of the
topbar slot, causing a visible layout shift.

Add explicit page-frame path that wraps both editing and non-editing
states in TopBar.Slot. Use PageFrameViewTitleWrapper (plain div) and
PageFrameGrowingInput (without fixed 40px height/font overrides) so
the topbar's default sizing applies. Legacy path is unchanged.

Refs DE-1132
Co-Authored-By: Claude <noreply@anthropic.com>
@linear-code

linear-code Bot commented Apr 20, 2026

Copy link
Copy Markdown

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 20, 2026
@JonasBa
JonasBa marked this pull request as ready for review April 20, 2026 22:10
@JonasBa
JonasBa requested a review from a team as a code owner April 20, 2026 22:10
Comment thread static/app/views/issueList/editableIssueViewHeader.tsx
@priscilawebdev

priscilawebdev commented Apr 21, 2026

Copy link
Copy Markdown
Member

I noticed that when editing a title now, it shifts a little:

Screen.Recording.2026-04-21.at.09.15.26.mov

@JonasBa

JonasBa commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

I noticed that when editing a title now, it shifts a little:

Screen.Recording.2026-04-21.at.09.15.26.mov

Excellent eye @priscilawebdev! I've went ahead and removed the dashed border, I think that is excessive of an indicator given that we already render the button next to the title

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a5ea497. Configure here.

Comment thread static/app/views/issueList/editableIssueViewHeader.tsx
@JonasBa
JonasBa merged commit 12f6c4c into master Apr 21, 2026
64 of 65 checks passed
@JonasBa
JonasBa deleted the jb/issues/editable-title-de-1132 branch April 21, 2026 20:38
@github-actions github-actions Bot locked and limited conversation to collaborators May 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants