Skip to content

Rename "Close" to "Apply" in DependencyPropertiesDialog#1634

Merged
vogella merged 1 commit intoeclipse-pde:masterfrom
ptziegler:issue1632
Mar 4, 2025
Merged

Rename "Close" to "Apply" in DependencyPropertiesDialog#1634
vogella merged 1 commit intoeclipse-pde:masterfrom
ptziegler:issue1632

Conversation

@ptziegler
Copy link
Contributor

This changes the name of the OK button to "Close", to avoid any confusion with the similarly named "Cancel" button.

Closes #1632

This changes the name of the OK button to "Close", to avoid any
confusion with the similarly named "Cancel" button.

Closes eclipse-pde#1632
@ptziegler
Copy link
Contributor Author

I think the buttons are sorted lexicographically, so the OK button is now the first button. This might mess with muscle memory, but I'm not sure if it's worth changing, especially because you have the same in e.g. the property pages.

image

@github-actions
Copy link

github-actions bot commented Mar 3, 2025

Test Results

   285 files  +   285     285 suites  +285   47m 9s ⏱️ + 47m 9s
 3 608 tests + 3 608   3 532 ✅ + 3 532   76 💤 + 76  0 ❌ ±0 
11 016 runs  +11 016  10 785 ✅ +10 785  231 💤 +231  0 ❌ ±0 

Results for commit 0da2139. ± Comparison against base commit 515f996.

@merks
Copy link
Contributor

merks commented Mar 4, 2025

The button order is different (opposite) on MacOs than on Windows/Linux. Buttons are not sorted...

https://stackoverflow.com/questions/6667271/standard-placement-order-for-common-buttons

@vogella
Copy link
Contributor

vogella commented Mar 4, 2025

I think the buttons are sorted lexicographically, so the OK button is now the first button. This might mess with muscle memory, but I'm not sure if it's worth changing, especially because you have the same in e.g. the property pages.

image

Did you test / develop on different platforms? As @merks said, button are OS specific

@ptziegler
Copy link
Contributor Author

Did you test / develop on different platforms? As @merks said, button are OS specific

The order is definitely OS specific. On Windows, the order is "Apply" "Cancel" but on Linux it's "Cancel" "Apply". Weird... 🤷

image

@vogella vogella merged commit 6949fca into eclipse-pde:master Mar 4, 2025
13 of 15 checks passed
@vogella
Copy link
Contributor

vogella commented Mar 4, 2025

Test / build failure seems unrelated.

Thanks @ptziegler

@akurtakov
Copy link
Member

Please don't merge things until we open the stream. I don't want such merges to break build due to missing version bumps or etc.

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.

Ambiguous button names in dependency properties dialog

4 participants