[CONTEDIT-136] Option to adjust LinkField width Created: 05/Jul/17 Updated: 11/Aug/17 Resolved: 11/Aug/17 |
|
| Status: | Closed |
| Project: | Content Editor |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | New Feature | Priority: | Neutral |
| Reporter: | Maxime Michel | Assignee: | Unassigned |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 5h | ||
| Time Spent: | 3d 2.5h | ||
| Original Estimate: | 3d | ||
| Attachments: |
|
||||||||||||||||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Story Points: | 5 | ||||||||||||||||
| Description |
|
The new look of 'browse' button implies that LinkFields do no longer need a 100% width. Create an option to adjust that. |
| Comments |
| Comment by Mikaël Geljić [ 18/Jul/17 ] |
|
hieu.nguyen I just attached a "cropped" version of Anja's mockups from
|
| Comment by Mikaël Geljić [ 25/Jul/17 ] |
|
As discussed upon review, we move that back to the content-editor for now. Main reason is that we don't feel like introducing configurable width in field definitions (nor in plain `LinkFieldDefinition`); while at the same time, we vaguely plan to make field-definitions less UI-specific (and leave sizing to a new layout configuration in the form instead). |
| Comment by Rasmus Skjoldan [ 07/Aug/17 ] |
|
I'm not entirely sure how it's going to look like without the adjusted width but it's not critical to get done. |
| Comment by Anja von Gunten [ 10/Aug/17 ] |
|
I will spend some time on finding an alternative solution fx. "using a smaller fixed-width, or recalculating the width only upon value-change with a very simple strategy (if blank, set small fixed width, if not empty set full width)" as Mika has suggested. If we come up with an easier solution to be implemented that significantly improves the current version it would be nice to have it done for the release. |
| Comment by Mikaël Geljić [ 11/Aug/17 ] |
|
This is now superseded by |