Opened 10 years ago
Last modified 10 years ago
#12466 confirmed New Feature
Editor removes links with no text
Reported by: | dan turcu | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 4.0 |
Keywords: | Cc: |
Description
If you put from the source mode a link that has no text the editor automatically removes it. This is bad because I am using css to define images to be put instead of the text and the link is something like <a href="..." class="myclass"></a> and in the myclass is defined the actual image to be clicked on.
Steps to reproduce:
- go to demo page
- go in source mode
- replace existing source with <a href="www.google.com"></a>
- exit the source mode
- enter again the source mode and you will see that the tag is no there anymore
The best solution I think would be that a config parameter to change this behavior would exist
Attachments (1)
Change History (5)
Changed 10 years ago by
Attachment: | _template.html added |
---|
comment:1 Changed 10 years ago by
Type: | Bug → New Feature |
---|---|
Version: | 4.4.3 → 4.0 |
comment:2 Changed 10 years ago by
Status: | new → confirmed |
---|
We have decided to confirm this ticket and have at least checked out.
Configuration option sounds reasonable but we need to first check how such element would behave in wysiwyg area and whether it will break anything.
At the moment such elements are removed from editor because they are not editable in wysiwyg mode.
This issue could be workaround by implementing such links as widgets.
comment:3 Changed 10 years ago by
Any news about this New Feature, it will be implemented, it's already implemented ?
I'm not sure what to think of that.
We don't allow usage of empty inline elements because there is no way to get to them in wysiwyg mode (they are invisible). That is why these elements are removed by default.
With
content
property however there is possibility to insert image into link and have link gain size and have link being clickable. Perhaps the idea with config option (e.g. don't remove elements with such class(es)) is not that bag.I'm changing this ticket to Feature Request and I'm very curious what you guys (CKSource team) have got to say.