Bug 496019 - Add tooling in the VSM editor to streamline the most common cases
Summary: Add tooling in the VSM editor to streamline the most common cases
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Properties (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.1.2   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks: 511783
  Show dependency tree
 
Reported: 2016-06-13 11:40 EDT by Pierre-Charles David CLA
Modified: 2017-02-06 12:30 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pierre-Charles David CLA 2016-06-13 11:40:43 EDT 
The most obvious case: setting up a reference widget to view & edit a specific ERerefence in a canonical way should only require choosing the EReference (among a list of proposals computed from the context).
Comment 1 Pierre-Charles David CLA 2016-09-12 08:58:01 EDT 
Moving to 4.1.1, but it's not clear if this is still relevant given the new widget introduced in bug #496018 and the VSM-editor-only ergonomics improvments that would be covered by bug #489293.
Comment 2 Pierre-Charles David CLA 2016-11-10 09:26:14 EST 
The EEF runtime expect non-empty expressions for widgets callbacks, and complains rather loudly if they are missing. On the VSM side, this corresponds to missing or empty "Begin" (InitialOperation elements), but the error message produced by EEF talks about "blank expressions", so users can not interpret it correctly.

We should probably automatically create "no-op" operations (with a simple "Change Context: var:self") wherever this can happen. It will avoid EEF errors, have no behavior change, and save a few clicks for specifiers.
Comment 3 Pierre-Charles David CLA 2016-11-24 09:55:51 EST 
Not really "tooling" per se, but in terms of ergonimics of the VSM editor, since 4.1 the "Identifier" section has been move to the "Avanced" tab. It's fine for most widgets, bug for pages, groups and custom widgets the identifier is more useful and should be more prominent.
Comment 4 Eclipse Genie CLA 2016-12-02 04:17:56 EST 
New Gerrit change created: https://git.eclipse.org/r/86211
Comment 6 Pierre-Charles David CLA 2016-12-02 09:38:56 EST 
For this iteration, the only thing that was done was to auto-create no-op navigation operations (Change Contex to var:self) everywhere an operation is needed by the metamodel when the user creates a new element. This way the callbacks are always present (preventing things like #498293), and more directly visible to the specifier (who otherwise has to know he has to clik on a context menu to create the callbacks).

For future iterations we'll create more focused tickets instead of general ones like this one.
Comment 7 Pierre-Charles David CLA 2016-12-08 11:12:49 EST 
Available in Sirius 4.1.2 (see https://wiki.eclipse.org/Sirius/4.1.2 for details).
Comment 8 Eclipse Genie CLA 2017-02-06 12:30:46 EST 
New Gerrit change created: https://git.eclipse.org/r/90445