[PAGES-245] Missing default value for component position in new component dialog Created: 25/Feb/20 Updated: 14/Sep/22 Resolved: 14/Aug/20 |
|
| Status: | Closed |
| Project: | Magnolia pages module |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.2.3 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Michael Duerig | Assignee: | Martin Haderka |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | 0.3h | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||
| Epic Link: | UI framework: depending issues | ||||||||||||||||||||||||
| Sprint: | UI FW 8 | ||||||||||||||||||||||||
| Description |
|
The add where? radio buttons do not select the default value (bottom).
cc apchelintcev, rkovarik |
| Comments |
| Comment by Michael Duerig [ 25/Feb/20 ] |
|
The default is not applied because the position property of that dialog is effectively transient (i.e. not stored in JCR) but the JcrPropertyDescriptor does not convey this fact and thus JcrItemInteractionStrategyImpl can't know whether this is a property with an empty value or no property at all. One solution we discussed in Slack is to explicitly mark transient properties by adding an isTransient member to ValueBoundProperty. |
| Comment by Šimon Demočko [ 13/Aug/20 ] |
|
An observation: Even though nothing is selected, one can "next" the dialog, it won't nag. It defaults into adding "after" |