silexlabs / Silex

Silex live web creation, free/libre no-code website builder, next gen Webflow for the static web
http://www.silex.me
GNU Affero General Public License v3.0
2.2k stars 580 forks source link

Tags for Image (V3) #1531

Closed GetLikha closed 12 months ago

GetLikha commented 1 year ago

Still working my way around alpha 93 of V3, and if you set a tag name for an image to something else, the option to set it back to IMG disappears. image image

Maybe it should be removed for certain elements? Only makes sense. Because on export, the IMG becomes the tag name you set. image

Also yeah the dropdown text is hard to see, maybe because I'm using dark mode?

GetLikha commented 1 year ago

So I digged a bit further, if you set a link, the IMG tag becomes an tag, instead of being wrapped around with an tag. image image

lexoyo commented 1 year ago

Oh that's a good one, thank you @GetLikha

When you set a link on an element, it does not wrap it into a link, instead it makes it an A tag. It is because from v3 silex philosophy is to be as close to html/css as possible

When the element is an image element, this breaks the image. Probably other elements too. Please note that when you remove the link it goes back to IMG (right?)

What we should probably do it to warn the user in the link editor and the tag selector?

For the color of the dropdown, could you check if it is your browser? Silex is supposed to be in dark mode already (no light mode yet)

GetLikha commented 1 year ago

Ah I see, so I should wrap the with a container that'll become a tag, that's actually a pretty neat way to teach people how html/css works.

I think just a pop up that warns the user like Warning: Changing this setting may cause the element to work incorrectly. then maybe a link to a docs about it.

As for the dropdown, well I'm not 100% sure, but the dropdown should still be styled correctly across all browsers right? I'm using Opera One.

lexoyo commented 1 year ago

Very true, I will have to do something about this in any case

As for the dropdown, well I'm not 100% sure, but the dropdown should still be styled correctly across all browsers right? I'm using Opera One.

Oh i'm not sure about opera... Hmm someone should investigate that as opera is an important actor of the free/libre open source world

Would you open an issue with the opera-related problems?

GetLikha commented 1 year ago

I'd love to do that, I'll use both Chrome and Opera and see if there's any bugs.

Also I'd love to contribute to docs, do you mind if I can create pages on how to use Silex?

lexoyo commented 1 year ago

I would love a collaborative effort on the docs

Let's discuss it in the forums? You probably already have access to create pages otherwise ask JP on the forums. Also in this issues there are useful information about how links work that we should use in the docs :)

lexoyo commented 1 year ago

Or just go for it if you feel like it! :)

lexoyo commented 12 months ago

I ended up removing the link option for images, that was the simplest to do Can you confirm it solves this issue please @GetLikha ?

lexoyo commented 12 months ago

It looks like this Screenshot from 2023-08-28 09-39-19

GetLikha commented 12 months ago

It works! I also wrapped it around a container and added a link, it works well

lexoyo commented 12 months ago

Great thx!