User interface: rename the "Process" button to "Processing"#176
Closed
douglas-carmichael wants to merge 1 commit into
Closed
User interface: rename the "Process" button to "Processing"#176douglas-carmichael wants to merge 1 commit into
douglas-carmichael wants to merge 1 commit into
Conversation
The lower toolbar button that opens the sample-processing options dialog was labelled "Process". As a bare imperative verb it reads like an action that starts processing the samples, which confused new users - especially since the actual run is started by the "Convert" button right above it. Rename it to "Processing", a noun that matches the title of the dialog it opens (IDS_PROCESSING_DIALOG) and parallels its neighbour "Settings"; both carry the same gear icon. Also add the Alt+P mnemonic its siblings have, and reword the tooltip to say it opens the options.
douglas-carmichael
added a commit
to douglas-carmichael/ConvertWithMoss
that referenced
this pull request
Jul 7, 2026
…ng rename (PR git-moss#176) - SoundFont 2: skip presets that reference no samples (the commercial-bank 'marker' presets like 'E-mu Systems 2007'), so they no longer become empty instruments. Mirrors the Ensoniq detector guard; new IDS_NOTIFY_SF2_SKIP_EMPTY_PRESET. - UI: rename the 'Process' button to 'Processing' (matches the dialog title, non-imperative, Alt+P) and reword its tooltip.
Owner
|
I changed this intentionally from Processing to Process since the longer name broke the layout on some platform/screen resolutions and I was unable to fix this. Until you checked this on all OSs and some resolutions I don't want to go through the hassle again. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The lower-toolbar button that opens the sample-processing options dialog is labelled "Process". As a bare imperative verb it reads like an action that starts processing the samples — which is confusing for new users, especially since the actual run is started by the "Convert" button directly above it, and this button only opens a configuration dialog.
Fix
Rename it to "Processing" — a single word that:
IDS_PROCESSING_DIALOG), so button and title now agree;Also adds the
Alt+Pmnemonic that its sibling buttons already have, and rewords the tooltip from "Configure processing of samples during conversion." to "Open the options for processing samples during conversion." to reinforce that it opens a dialog.Resource-only change (
Strings.properties); the button wiring inMainFrameis untouched (the"Process"argument there is theProcess.pngicon filename, not the label). CHANGELOG updated.