-
-
Notifications
You must be signed in to change notification settings - Fork 70
friendly message for "cannot change mode" #189
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
Comments
Created by: villares
We have seen behaviour like what @micuat proposed before, like creating a folder when one is missing and just telling the user! |
Created by: benfry After looking through this for a bit, there just isn't an error message that makes sense here and will be useful enough. And there are too many edge cases to really make it a good experience for users. So for alpha 6, I'm changing it to just open a new window in situations where things don't match—i.e. there have been changes, the code would be incompatible, etc. In those cases, the user will know what they want to do (i.e. copy and paste code, or close the old window w/o saving, or whatever) and the mild inconvenience is outweighed by the complication of trying to anticipate all possible situations and automatic resolutions (that may not work, and may not be what the user wanted anyway). |
Created by: github-actions[bot] This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs. |
Created by: micuat
Description
(not sure to put this here or processing3)
The dialog is quite "unfriendly" at the moment and suggest to change
processing4/build/shared/lib/languages/PDE.properties
Lines 588 to 589 in 4153e4f
from https://discourse.processing.org/t/i-cant-change-mode/28732/5
Expected Behavior
something like "please create a new sketch to change mode" ?? or what is even better is the mode change action will automatically open a new window
Current Behavior
Cannot change mode,\nbecause "%s" mode is not compatible with current mode.
The text was updated successfully, but these errors were encountered: