-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
UX : Browser strange toggle buttons #17827
Comments
The Show lines number count for all browsers option still works, so do we turn it by default or ask people before ? |
I think it is better to ask ask people about it. I turn it on by default on all my images with a simple script in the pharo launcher: |
Okay thanks, and @Nyan11 does it make sense to you to put a check box even for the modify protocol icon? It opens a menu so it looks a little weird to me |
The bottom buttons are separted in two categories:
Issue 1 - Seperate the categoriesFirst, we should add a separator between them, to show the distinction between the 2 categories of buttons. Issue 2 - Remove the cross buttonThere is a small button that remove the protocole or the package extension. I think it is useless because the same effect could be achieve by clicking either the protocol or extension label. Issue 3 - Put the protocole label inside the protocole edit button.We can see on the hover that the button is separated from the label. It would be better if the label was inside the button. Issue 4 - Have the same type of button for the browser / code options.Currently, the Format has a small checkbox while the two others change their labels. Issue 5 - Have explicit names for the browser / code options.Put explicit names like |
For me it makes sense. The protocole is either a protocol or an extension. So it can be represented with a checkbox (binary choice). |
For Issue 3, if we keep the current protocol and add the label 'Classify Method' before it would look strange, does having "Classify method, protocol/package is : currentProtocol" looks good to you? |
I found it too verbose. but i am not sure how it can be reduce. |
Alexis pay attention in english there is no space before : BTW a shorter version can be Classified in protocol: building |
@Nyan11, regarding your comment saying you “turn it on by default on all my images with a simple script in the pharo launcher”: you could also use ‘Save to Disk’ as shown on the following screenshot, the setting is then loaded in other images automatically (see |
In the bottom right there are 5 buttons:
From: #17812 (comment)
@jecisc
@Rinzwind
The text was updated successfully, but these errors were encountered: