Rename the newTabPosition strings #18038

Closed
opened 2026-01-31 06:02:02 +00:00 by claunia · 3 comments
Owner

Originally created by @carlos-zamora on GitHub (Jul 25, 2022).

Follow-up from #13421

Settings UI

Position of the new tab --> Position of newly created tabs
Specifies position of the new tab --> Specifies where new tabs appear in the tab row
At the end --> After the last tab
After current tab --> After the current tab

JSON

atTheEnd --> afterLastTab
afterCurrentTab --> afterCurrentTab

No need for legacy converter

Code

we should rename the enums to match the new names

Originally created by @carlos-zamora on GitHub (Jul 25, 2022). Follow-up from #13421 ### Settings UI Position of the new tab --> Position of newly created tabs Specifies position of the new tab --> Specifies where new tabs appear in the tab row At the end --> After the last tab After current tab --> After the current tab ### JSON atTheEnd --> afterLastTab afterCurrentTab --> afterCurrentTab No need for legacy converter ### Code we _should_ rename the enums to match the new names
Author
Owner

@serd2011 commented on GitHub (Jul 26, 2022):

Is this a legacy converter? It doesn't compile for me without it. 6c2316df44/src/cascadia/TerminalSettingsModel/TerminalSettingsSerializationHelpers.h (L240-L246)

@serd2011 commented on GitHub (Jul 26, 2022): Is this a legacy converter? It doesn't compile for me without it. https://github.com/microsoft/terminal/blob/6c2316df44779deb9480c769e3354187660e6736/src/cascadia/TerminalSettingsModel/TerminalSettingsSerializationHelpers.h#L240-L246
Author
Owner

@DHowett commented on GitHub (Jul 26, 2022):

Sorry, we didn't make that super clear. A legacy converter is one that includes support for the old data that might be in a user's settings file 😄

@DHowett commented on GitHub (Jul 26, 2022): Sorry, we didn't make that super clear. A legacy converter is one that includes support for the old data that might be in a user's settings file :smile:
Author
Owner

@ghost commented on GitHub (Sep 13, 2022):

:tada:This issue was addressed in #13602, which has now been successfully released as Windows Terminal Preview v1.16.252.🎉

Handy links:

@ghost commented on GitHub (Sep 13, 2022): :tada:This issue was addressed in #13602, which has now been successfully released as `Windows Terminal Preview v1.16.252`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.16.252) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#18038