-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cannot delete link info from basic Tile #397
Comments
how are you thinking to fix this issue? |
The most basic solution would I think be to have a 'delete link' button on the tile configuration page in the compose popup for the basic tile and body text tile. An extra feature would be to show the link that is stored on the tile compose popup after an item has been dropped (near to the delete button described above). |
I think it would be probably better to add a button to clear all contents on the tile (similar to the ones on list tiles); this could be implemented on the base tile. |
No, I wouldn't put it on a clear button on the tile itself as with the list tile, interaction wise. With the list tile you drag/drop and order the items in the tile itself without going into to a config popup, but the only information you can throw away is the link. (what you see is what you delete). The basic tile allows overriding the title and description, uploading an extra image. If you put the 'clear' action of the basic tile on the main compose screen it might not be obvious to users that they are throwing away custom modifications they or someboddy else made but didn't see and they cannot restore that work easily (by 'redropping the item'). Therefore I'd place a delete button on the page/panel were the info is displayed, providing the user more context. I'd opt to keep the display of tiles in the compse tab as clean as possible. |
ok, I agree with you. |
I'm currently reviewing this issue and I found the following:
IMO, we must allow editing URL on basic and list tiles; carousel tile don't make sense. I'll open another issue to address the list tile. |
Pfeww that's an old issue. :-) The carousel tile has been changed quite a bit since this issue was started, the customisation of every carousel item's fields after dropping was added somewhere 2016 by @frapell IIRC. That's why carousel has evolved away from List Tile. It used to work only by reference. |
When you drop a content item on a basic tile, the content is 'connected' to the basic tile and its title and description is copied to the basic tile fields. It is also possible to edit a basic title with a tile and description and not link any content.
On the compose tab it is possible to override the Title and description and it is possible to add an image, but you cannot 'disconnect' the dropped content item from step one. This means that:
When you separate access to the compose tab (content editor domain) from access to the layout tab (site manager) and use templates to set up pref-configured forms, it is necessary that content editor can 'clean' a basic tile.
Exactly the same concern has the body text tile. If you don't use it, it stays empty, if you drop something on it, it is not possible as an editor to clear the tile again.
The text was updated successfully, but these errors were encountered: