Skip to content
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

Rename buttons in disk_changed dialog #57062

Closed

Conversation

wachtel1
Copy link

@wachtel1 wachtel1 commented Jan 22, 2022

"The following files are newer on disk" dialog buttons:
Resave → Save Local Changes
Reload → Reload From Disk (Discard Local Changes)
Cancel → Do Nothing

Bugsquad edit: This closes godotengine/godot-proposals#3456.

"The following files are newer on disk" dialog buttons:
Resave → Save Local Changes
Reload → Reload From Disk (Discard Local Changes)
Cancel → Do Nothing

godotengine/godot-proposals#3456
@wachtel1 wachtel1 requested a review from a team as a code owner January 22, 2022 15:35
@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:editor usability labels Jan 22, 2022
@Calinou Calinou added this to the 4.0 milestone Jan 22, 2022
editor/editor_node.cpp Outdated Show resolved Hide resolved
@akien-mga
Copy link
Member

Can you post a screenshot?

I'm afraid this makes quite lengthy buttons, even more so with some translations where sentences are typically longer than English (French, German).

@KoBeWi
Copy link
Member

KoBeWi commented Apr 8, 2022

I think the "(Discard Local Changes)" part is not necessary, but the dialog is big enough:
image

@akien-mga
Copy link
Member

Maybe it can be shortened to "Reload (Discard Local Changes)"?

The proposal also had interesting suggestions to reword the intro paragraph, which I think would be good to take into account to make this even more explicit. The buttons should be terse, but the paragraph in a huge window can be a bit more detailed.

In the proposal you also mentioned changing the dialog title, which I think would make sense.

@wachtel1
Copy link
Author

Sorry for leaving this unattended for so long. I can't install proper C++ tooling on my machine, so building from the source is not available for me right now.

Maybe like this?

Title:

Files Have Been Changed

Text:

The following files currently opened have changes made by an external program.
Do you want to save changes from the Godot editor or reload the files from the disk and discard them?
This action can't be undone.

Buttons:

Save Local Changes
Reload (Discard Local Changes)
Do Nothing

@akien-mga
Copy link
Member

Sounds good to me, maybe just a tiny edit to the text: discard them -> discard changes (since "them" can refer to "files" and not "changes" in that sentence).

For the title, maybe "Files Changed on Disk"?

@YuriSizov YuriSizov modified the milestones: 4.0, 4.1 Feb 9, 2023
@YuriSizov YuriSizov modified the milestones: 4.1, 4.2 Jun 14, 2023
@AThousandShips AThousandShips modified the milestones: 4.2, 4.3 Oct 26, 2023
@JoNax97
Copy link
Contributor

JoNax97 commented Jun 1, 2024

This can probably be closed in favor of #90660

@AThousandShips AThousandShips changed the title Renamed buttons in disk_changed dialog Rename buttons in disk_changed dialog Jun 1, 2024
@AThousandShips
Copy link
Member

AThousandShips commented Jun 1, 2024

Superseded by:

If any further changes are desired please open a new PR with those changes separated

Thank you for your contribution nonetheless!

@AThousandShips AThousandShips added archived and removed cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Jun 1, 2024
@AThousandShips AThousandShips removed this from the 4.3 milestone Jun 1, 2024
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.

Make a dialogue window "The following files are newer on disk" easy to understand
7 participants