Skip to content

fix(custom-views): editableTabTitle Improvements #76247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

MichaelSun48
Copy link
Member

This component makes some important improvements to the editableTabTitle component, which is used in custom views to allow users to rename the titles of their tabs. These fixes include:

  1. The text within the tab no longer wraps weirdly, shifting inside and outside the container upon every new character
  2. You can now type spaces inside the text box
Screen.Recording.2024-08-14.at.5.27.48.PM.mov

@MichaelSun48 MichaelSun48 requested a review from malwilley August 15, 2024 00:28
@MichaelSun48 MichaelSun48 requested a review from a team as a code owner August 15, 2024 00:28
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 15, 2024
@MichaelSun48 MichaelSun48 merged commit 592f367 into master Aug 15, 2024
42 checks passed
@MichaelSun48 MichaelSun48 deleted the msun/fixEditableTabTitle branch August 15, 2024 18:25
MichaelSun48 added a commit that referenced this pull request Aug 16, 2024
…lag (#75883)

This PR releases the custom views feature into the issue stream under a
feature flag. Flagged users will also see the "Custom Search" button has
been removed from the issue stream in favor of custom views. The bulk of
this new component exists in `<CustomViewsIssueListHeader/>`, whose
parent component is in overview.tsx.

Several changes have also been made to the base draggable tab components
to iron out any compatibility issues that did not appear during sandbox
development.

Things that are still broken that I am actively working on: 
* ~Tab Renaming is both ugly and broken (can't type in spaces)~ (#76247)
* Tab overflow logic does not exist at all 
* QueryCounts do not work for any tabs with queries that do not match
queries of previous default tabs
* Some queries that are functionally the same but off by a space
register as different, thus triggering the "unsaved changes" state.
* Tabs do not animate in when added, nor do they animate out when
deleted
* No loading or failure state for tabs request 
* ~Stories is busted as a result of all of the changes made to base tabs
components~ (deleted stories, this component is not, and was not meant
to be reusable)
@github-actions github-actions bot locked and limited conversation to collaborators Aug 31, 2024
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.

2 participants