{"action":"created","issue":{"url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder/issues/313","repository_url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder","labels_url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder/issues/313/labels{/name}","comments_url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder/issues/313/comments","events_url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder/issues/313/events","html_url":"https://github.com/kasemir/org.csstudio.display.builder/issues/313","id":277340708,"number":313,"title":"Problems editing properties of multiple selected widget.","user":{"login":"claudio-rosati","id":10833922,"avatar_url":"https://avatars1.githubusercontent.com/u/10833922?v=4","gravatar_id":"","url":"https://api.github.com/users/claudio-rosati","html_url":"https://github.com/claudio-rosati","followers_url":"https://api.github.com/users/claudio-rosati/followers","following_url":"https://api.github.com/users/claudio-rosati/following{/other_user}","gists_url":"https://api.github.com/users/claudio-rosati/gists{/gist_id}","starred_url":"https://api.github.com/users/claudio-rosati/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/claudio-rosati/subscriptions","organizations_url":"https://api.github.com/users/claudio-rosati/orgs","repos_url":"https://api.github.com/users/claudio-rosati/repos","events_url":"https://api.github.com/users/claudio-rosati/events{/privacy}","received_events_url":"https://api.github.com/users/claudio-rosati/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-11-28T10:55:20Z","updated_at":"2017-11-28T14:58:22Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"- Select a label widget A with class DEFAULT. You can change the foreground colour.\r\n- Select a label widget B with class TITLE. You cannot change the foreground colour.\r\n- Now select first B and then add A to the selection. You cannot change the foreground colour.\r\n- Now select first A and then add B to the selection. You can now change the foreground colour of both labels.\r\n\r\nI think that if a property has at least one widget locking it because of a class, that property should not be editable.\r\n\r\nMoreover, if the value is not equal in all the selected widgets, we should change the property name colour to blue (for example), to let the user now that the displayed value is not identical on all selected widgets."},"comment":{"url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder/issues/comments/347549490","html_url":"https://github.com/kasemir/org.csstudio.display.builder/issues/313#issuecomment-347549490","issue_url":"https://api.github.com/repos/kasemir/org.csstudio.display.builder/issues/313","id":347549490,"user":{"login":"kasemir","id":1932421,"avatar_url":"https://avatars1.githubusercontent.com/u/1932421?v=4","gravatar_id":"","url":"https://api.github.com/users/kasemir","html_url":"https://github.com/kasemir","followers_url":"https://api.github.com/users/kasemir/followers","following_url":"https://api.github.com/users/kasemir/following{/other_user}","gists_url":"https://api.github.com/users/kasemir/gists{/gist_id}","starred_url":"https://api.github.com/users/kasemir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kasemir/subscriptions","organizations_url":"https://api.github.com/users/kasemir/orgs","repos_url":"https://api.github.com/users/kasemir/repos","events_url":"https://api.github.com/users/kasemir/events{/privacy}","received_events_url":"https://api.github.com/users/kasemir/received_events","type":"User","site_admin":false},"created_at":"2017-11-28T14:58:22Z","updated_at":"2017-11-28T14:58:22Z","author_association":"OWNER","body":"Yes, that's how it currently behaves.\r\nThe first widget that you select determines the values shown in the property list, including the class behaviour.\r\nWhen then selecting additional widgets, the property list is narrowed down to the common properties, but still showing the values of the first selected widget.\r\nWhen now setting a property, the new value is applied to all.\r\n\r\nShowing the values of the first selected widget is important, because that allows you to apply a selected value from one widget (the one selected first) to many widgets (the other selected ones) by simply pressing 'enter' on that value.\r\nOther tools (Power Point, LibreOffice Draw) would simply not show any value in that case, so then you need to first select just one item, make a mental note of the value, then manually enter that when you select the other items.\r\nSo I really like how it shows the value of the first selected widget. Somehow indicating that this value does not apply to every selected widget seems reasonable. Doing that via the label color would be a larger change to the code. Doing that via the color of the value would be easier, because all the *Binding* classes already know the value widget. Would also make more sense, because you want to indicate that there's something special about the _value_, it doesn't apply to all the selected widgets."}}
{"actor":{"display_login":"kasemir"}}