Skip to content

-[bug 949] #949 its reported in the issue that the save button in la…#970

Merged
mattrose merged 1 commit intognome-terminator:masterfrom
vssdeo:949-Fix-the-Save-button-behavior-for-Layouts
Jan 12, 2025
Merged

-[bug 949] #949 its reported in the issue that the save button in la…#970
mattrose merged 1 commit intognome-terminator:masterfrom
vssdeo:949-Fix-the-Save-button-behavior-for-Layouts

Conversation

@vssdeo
Copy link
Copy Markdown
Contributor

@vssdeo vssdeo commented Dec 11, 2024

Its reported in the issue that the save button in layout overrides the profile, custom command, working dir. The same does not happen when close button is pressed or pref window is closed. In my observation,I have seen that the profile is overwritten, but custom command and wording dir are updated.

The updation of working dir and custom command when editing without save button seems a bit different but I haven't changed that part for now.

For sake of consistency I have ensured the updation of profile (which anyways gets updated from drop down menu) but is over written by save button, happens when save button is pressed.

…ve button in layout overrides the profile, custom command, working dir. The same does not happen when close button is pressed or pref window is closed. In my observation,I have seem that the profile is overwritten, but custom command and wording dir are updated.

The updation of working dir and custom command when editing without save button seems a bit different but I haven't changed that part for now.

For sake of consistency I have ensured the updatation of profile (which anyways gets updated from drop down menu) but is over written by save button, happens when save button is pressed.
@mattrose
Copy link
Copy Markdown
Member

Looks good, sorry about the wait.

@mattrose mattrose merged commit 8cda0ca into gnome-terminator:master Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants