Updated the property's look and feel #7
Conversation
TRexStark
commented
Sep 1, 2020
TRexStark
commented
- Added some styles to make the property editor look close to the Umbraco media picker property
- Changed the "Add" button to become the link-square of the picker
- Changed the "Remove" link to the cross look and feel and positioned it above and on top of the preview box
- Re-positioned the sort icon/button to the top left of the preview box
- Reduced the size of the preview box to the same height as the media picker property but the width at a ratio of an HD size proportionally
- made all preview boxes as well as the "Add" button to align vertically
- New styles are referenced from a new stylesheet which has been called from the package.manifest
- Added some styles to make the property editor look close to the Umbraco media picker property - Changed the "Add" button to become the link-square of the picker - Changed the "Remove" link to the cross look and feel and positioned it above and on top of the preview box - Re-positioned the sort icon/button to the top left of the preview box - Reduced the size of the preview box to the same height as the media picker property but the width at a ratio of an HD size proportionally - made all preview boxes as well as the "Add" button to align vertically - New styles are referenced from a new stylesheet which has been called from the package.manifest
|
Hi @TRexStark Thanks for the PR. I will try to find sometime this week to review it. Stay tuned. Dave |
|
Hi @TRexStark Just wanted to let you know that I am a bit swamped with work lately, but I still have this on my radar. Will try to review this one later this week. Dave |
|
No worries @dawoe I'm planning to submit another Pull Request, I'm just testing it on a client site and will compile it hopefully later this week. |
|
Hi @TRexStark First of all the updates look great. But I noticed some small issues when using it as a single picker.
It still shows the sorting icon and it shows a empty square box when a video is picked Could you have a look at that ? Dave |
|
Is there a way to detect if it's in single mode in the view? A added class when in singlemode or something? |
I will look into that no worries. |
Based on the feedback "dawoe#7 (comment)", the following updates have been made: - Hide additional box container of the button when "Allow Multiple" is not toggled - Hide the Sort Icon when "Allow Multiple" is not toggled The items above will actually not exist (using ng-if instead of ng-hide) for single items
Hi @dawoe |
Used ng-hide for the picker btn instead of ng-if as the button would disappear entirely when the item is removed
|
Hi @TRexStark This looks great. I will merge this one and release a new version of the package |
