https://twill.io logo
Join Discord
Powered by
# 🖼twill-image
  • m

    mateoD

    08/05/2022, 3:47 PM
  • m

    mateoD

    08/05/2022, 3:48 PM
    position relative is the problem
  • i

    ifox

    08/05/2022, 4:17 PM
    you can publish the view @mateoD
  • m

    mateoD

    08/08/2022, 7:08 AM
    @ifox if i remove from inspect position relative image displays as well
  • m

    mateoD

    08/08/2022, 8:42 AM
    $(".twill-image-wrapper").css({ 'position' : '' }) this works but I wanted to know if there is any other way to fix it from the configurations in twill-image
  • i

    ifox

    08/08/2022, 8:45 AM
    Same answer, you can publish the view
  • i

    ifox

    08/08/2022, 8:46 AM
    but i think if you have to remove that it's probably because you did something else wrong
  • i

    ifox

    08/08/2022, 8:47 AM
    It is meant to be there so that the image can be absolutely placed in that container, as well as the LQIP. Try it without the rest of your own css to see if there might be a conflict.
  • p

    patrick

    08/11/2022, 11:26 AM
    Hi @mateoD , in general it should be best to leave the twill-image css as-is for the reasons @ifox mentions above. It's probably a css rules in your container element that is causing a weird display (I see from the capture those are absolue position). I find it best to always wrap twill-images in a container element (div) to set the desired width. Often a around the image will make sure you image takes the full width of the parent container.
  • m

    mateoD

    08/30/2022, 1:04 PM
    hello everyone , how can I get the "alt text"of an image that I render $item->image('image');
  • u

    23r01nf1n17y

    08/30/2022, 1:42 PM
    $item->imageCaption(role: 'image')
  • u

    23r01nf1n17y

    08/30/2022, 1:43 PM
    $item->imageAltText(role: 'image')
  • m

    mateoD

    08/30/2022, 1:59 PM
    it works thank you
  • d

    dedli

    09/16/2022, 10:14 AM
    Hi ) How can I set compressed and crop image webp as backgroud? I read documentation couple times but I cannot understand. Maybe I missed something
    u
    • 2
    • 6
  • u

    23r01nf1n17y

    10/22/2022, 5:49 PM
    Hi How can I set compressed and crop
  • a

    Aidas (KasValgyt)

    03/22/2023, 7:41 AM
    Hi, I am super happy with this package, but can we use it without twill? Would love to have this option. Would it be hard to do it? This package already has static image support, so it can be used without Blocks or Media classes from base twill package. So maybe we could just move "area17/twill": "2.*|3.*", dependency to suggested composer package?
  • i

    ifox

    03/22/2023, 10:15 PM
    Hi @Aidas (KasValgyt) that's an interesting idea, and I agree there's a lot of value one could get out of the package even without Twill
  • i

    ifox

    03/22/2023, 10:16 PM
    I feel like it might be best to separate in another package that's fully agnostic to twill
  • i

    ifox

    03/22/2023, 10:16 PM
    I'll have a chat with @patrick about it
  • p

    patrick

    03/23/2023, 11:05 AM
    Hi @Aidas (KasValgyt), I had a look yesterday and the StaticImage class is relying on the other Image class that uses Twill models (Media) to built the image arrays.
  • p

    patrick

    03/23/2023, 11:06 AM
    StaticImage is creating "in-memory" Twill Media along with crops and passing that to the Image class.
  • p

    patrick

    03/23/2023, 11:08 AM
    I do like the idea of having a twill-image that can be used without Twill, but I think it would be quite a refactor.
  • Static image handling
    a

    Aidas (KasValgyt)

    03/24/2023, 10:49 AM
    Got it, thanks for all the input. We will give it a try internally and we will let you know if it works out. Hope it does as there are no similar packages out where.
    p
    • 2
    • 5
  • https://github.com/area17/twill-image/pull/53 Please please please 🙏
    a

    Afting

    04/04/2023, 1:33 PM
    https://github.com/area17/twill-image/pull/53 Please please please 🙏
  • hi <@145236772580884481> thanks! I assume you tested this on your laravel 10 app?
    i

    ifox

    04/04/2023, 1:34 PM
    hi @Afting thanks! I assume you tested this on your laravel 10 app?
  • Yes, haven't run into any problems so far
    a

    Afting

    04/04/2023, 1:56 PM
    Yes, haven't run into any problems so far
  • there you go 🙂 https://github.com/area17/twill-image/releases/tag/v1.0.4
    i

    ifox

    04/04/2023, 2:30 PM
    there you go 🙂 https://github.com/area17/twill-image/releases/tag/v1.0.4
  • c

    Constant Variable

    05/22/2023, 6:56 AM
    Images Alt text Hello, something missing in our opinion is the fact that if the ALT text is left empty, this one is not present in the
    <img>
    attributes, but for accessibility reasons it should be present BUT empty :
    alt=""
    It means that the image is 'decorative': https://www.w3.org/WAI/tutorials/images/decorative/
  • i

    ifox

    05/22/2023, 4:54 PM
    Hi @Constant Variable what you're asking would mean that the package considers that all images without an alt text filled out are decorative. Maybe that's correct, but then other people are going to say the opposite. Trust me we have a lot of experience working with accessibility consultants and there are a lot of subjective takes on these things. The optimal solution would be an explicit checkbox in the CMS to indicate that the image is decorative, or a configuration in the package to make that decision.
  • c

    Constant Variable

    05/22/2023, 5:23 PM
    Hé @ifox , thanks for your quick answer as usual 😉 Yes I know that it can make debate, here in Belgium, we have an association who checks our websites when we work for institutions, according to them an
    alt
    attribute is always necessary, even empty. https://www.anysurfer.be/en/documentation/articles/detail/techniques-to-add-an-alternative-text-to-an-image So yes, I think a package configuration option could be a good idea.