Skip to content

Finalize New Inline Rename UI#59351

Merged
ryzngard merged 3 commits into
dotnet:features/rename_ui_reworkfrom
ryzngard:features/rename_ui_work/new_ui
Feb 24, 2022
Merged

Finalize New Inline Rename UI#59351
ryzngard merged 3 commits into
dotnet:features/rename_ui_reworkfrom
ryzngard:features/rename_ui_work/new_ui

Conversation

@ryzngard
Copy link
Copy Markdown
Contributor

@ryzngard ryzngard commented Feb 7, 2022

This is the final spec for the rename UI. Shown are the two states: collapsed and expanded.

Expanded

image

@ghost ghost added the Area-Infrastructure label Feb 7, 2022
@ryzngard ryzngard force-pushed the features/rename_ui_work/new_ui branch 3 times, most recently from 23afe4e to f663b6f Compare February 7, 2022 23:16
@ryzngard ryzngard changed the title Finalize UI Finalize New Inline Rename UI Feb 7, 2022
@ryzngard ryzngard marked this pull request as ready for review February 8, 2022 01:58
@ryzngard ryzngard requested a review from a team as a code owner February 8, 2022 01:58
@ryzngard ryzngard requested review from a team and JoeRobich February 8, 2022 01:58
@davidwengier
Copy link
Copy Markdown
Member

Boy that really is very collapsed...

(you forgot to paste in the image)

@ryzngard
Copy link
Copy Markdown
Contributor Author

ryzngard commented Feb 8, 2022

Boy that really is very collapsed...

(you forgot to paste in the image)

Currently fighting build errors locally and couldn't run it again :( collapsed state is more a testament of myself than the adornment.

@ryzngard ryzngard requested review from a team as code owners February 8, 2022 19:29
@ryzngard ryzngard force-pushed the features/rename_ui_work/new_ui branch from 55286eb to 3124503 Compare February 22, 2022 23:00
@ryzngard ryzngard enabled auto-merge (squash) February 24, 2022 19:54
@ryzngard ryzngard merged commit f198b0d into dotnet:features/rename_ui_rework Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants