Skip to content
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

Closed
fredvd opened this issue Feb 26, 2014 · 7 comments · Fixed by #834
Closed

Cannot delete link info from basic Tile #397

fredvd opened this issue Feb 26, 2014 · 7 comments · Fixed by #834
Assignees
Labels

Comments

@fredvd
Copy link
Member

fredvd commented Feb 26, 2014

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:

  • The title will still link to the content item
  • Other linked content fields like the tags if enabled will still show on the field.

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.

@hvelarde
Copy link
Member

how are you thinking to fix this issue?

@fredvd
Copy link
Member Author

fredvd commented Mar 6, 2014

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).

@hvelarde
Copy link
Member

hvelarde commented Mar 6, 2014

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.

@fredvd
Copy link
Member Author

fredvd commented Mar 10, 2014

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.

@hvelarde
Copy link
Member

ok, I agree with you.

@hvelarde
Copy link
Member

I'm currently reviewing this issue and I found the following:

  • banner and carousel tiles are the only ones that allow editing the URL
  • basic, collection and list don't allow that (a little weird as carousel tile inherits from list tile)

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.

Banner Tile

Basic Tile

List Tile

Carousel Tile

Collection Tile

@fredvd
Copy link
Member Author

fredvd commented Sep 27, 2018

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants