Slick Image Gallery

[Johani (Joe)]

This has to do with the aspect ratio of your images. Slick always maintains the original aspect ratio of the image.

The height of each gallery is determined by the height of the tallest image in it. The width of the gallery is always the full width of the post.

The images you add are slotted in and centred both vertically and horizontally in the gallery. This prevents the page content from moving around when you navigate between images of different heights and keeps the navigation buttons fixed in one place.

You can control the color of the background with:

.slick-track {
    background: black;
}

[Johani (Joe)]

@Francois_Douville

Is it possible to display another icon? My community thinks they have to display slick even if they have 1 image.

@Francois_Douville

Is it normal that the text is not displaying?

Capture d’écran 2018-05-19 à 09.16.44.png718x202 19.1 KB

Crosspost

Tiles Image Gallery

Yes, but that’s not how it’s supposed to work. If you upload a group of images together, then there’s no line-breaks between them. I will definitely look into ways to prevent this from happening, but the best thing I can recommend now is to create a tutorial topic on your site if your users are having such a big problem with it. I pushed updates to Tiles / Slick, so, please update the themes. The update ensures that they work with the image LazyLoad theme which I recommend you use if your post…

---

I took a look at the translation issue @Francois_Douville I have a temporary workaround for now. You need to make a small edit to the theme. Go to the </head> section and look for this:

You need to change en to fr

So

  I18n.translations.en.js.composer.tiles_add_images_prompt = settings.Tiles_add_images_prompt;
  I18n.translations.en.js.tiles_gallery_button = settings.Tiles_button_text;

becomes:

  I18n.translations.fr.js.composer.tiles_add_images_prompt = settings.Tiles_add_images_prompt;
  I18n.translations.fr.js.tiles_gallery_button = settings.Tiles_button_text;

That’s all you need to change.

Because this is a local edit, you need to reapply it after every theme update, but it’s not a big problem because you only need to change two letters.

I will keep looking for a better way to integrate translations into themes and will keep you posted.

[Johani (Joe)]

Tiles / Slick got a tiny update today that deals with the translation issue. They will now use the awesome locale check described in @simon’s great post here

The locale for the translated strings will automatically be set and you will not need to mess around with the theme code after this update.

[jamesmarkcook (James Cook)]

This is refusing to work on v2.2.0.beta3 +130

My images are larger than the maximum allowed dimensions, I get the gallery preview but in the actual post the gallery isn’t displayed, just normal images. Could you confirm that this works on the version I have installed? Thanks

[Johani (Joe)]

Sure, I checked and everything seems to be OK on v2.2.0.beta3 +136

@jamesmarkcook

just normal images

Do you mean lighboxes or just images? Slick only targets lighboxes so that would explain why it’s not working for you. If that’s the case, It sounds like sidekiq is paused / not running on your site. Try checking

your.domain.com/sidekiq

[jamesmarkcook (James Cook)]

Thanks for the reply. There was an unknown issue on my site that was causing lightboxes to take a few minutes to generate, and it was this that was causing this plugin not to work. All seems to be working now

[cwmke (Henry W)]

@Johani I’m wondering why this component calls slick.min.js from cloudflare instead of being packaged with the component?

[Johani (Joe)]

@cwmke

I’m wondering why this component calls slick.min.js from cloudflare instead of being packaged with the component?

At the time, the Discourse theme system was a bit different.

Scripts added to a theme / component used to be loaded inline in the <head> of the document. It made a lot more sense to load it from a CDN.

Discourse themes now handle scripts much better thanks to @xrav3nz’s work

https://github.com/discourse/discourse/commit/6acdea37c4e77723c774152e9076dfedda99493a

scripts are now pulled into an external theme.js file.

So, it now makes sense to bundle slick into the theme instead of loading it from a CDN.

I just pushed a small update for that

[cwmke (Henry W)]

Thanks for the explanation and the quick fix!

[Globulle]

Hi,

For an obscure reason I cannot achieve the expected slideshow. My code is :

<div data-theme-slick="1">

![IMG_1903|640x480](upload://p1oiiQ1oeNUOYY7nNETBuerrduz.jpeg)
![IMG_1923|640x480](upload://il3ESwgSA3szneJ0y0S92FEISwr.jpeg) 
![P1310402_small|666x500](upload://4H0qqhqtH8W3VA0XHxjKgXZO332.jpeg) 
![IMG_1927|640x480](upload://1QICzfj34S6zS963xb9JQ8PAJkK.jpeg)
![IMG_1958|640x480](upload://8XLYdnbXaKasrjEhyhXaPvIGNHK.jpeg)
![P1310413_small|666x500](upload://1egsIAMlC7RwlKqTGghS5wsUjfz.jpeg) 
![IMG_1996|375x500](upload://e5HXMa7aYAnPlP3JHrI4PQTgjjf.jpeg) 

</div>

I obtain :

• the 1st two images outside the slideshow
• the 3rd, 6th and 7th images are in a slideshow without the 4th and 5th (don’t show up at all)

I assume this is linked to the different sizes of images… so is there a workaround other than changing the size of the source pictures? It seems that changing the size parameter in the code does not improve things.

Any help would be appreciated!

EDIT : now I’ve tried with all images reduced to the same size : they are all displayed one below the other, and no slideshow at all…

[Johani (Joe)]

Thank you for the detailed feedback

The issue you’re seeing occurs because you’re using small images. The images you’re using are not large enough for Discourse to generate lightboxes.

There are two relevant settings for this

max_image_height which defaults to 500px

max_image_width which defaults to 690px.

Discourse looks at images uploaded in posts and if they exceed the demisions set in either of those settings, it creates lightboxes for them.

Now, you’re free to change those the values of those settings on your site to force Discourse to generate lightboxes for smaller image; however, I would not recommend doing that.

That setting is also linked to the max deminsions an image can be displayed at, so if you change max-width to say… 500px, then the maximum width any image will display at in a post will also be 500px - the same applies to changing the max-height setting.

What I would recommend is to use larger images that exceed either the max-height setting or the max-width setting. The component would then work with those images without any issues.

Let me know if the problem persists.

[elGrande]

After the last update now the arrows are missing:

Desktop:

Mobile:

[ckshen]

Am I the only one who see two of the same icons for the Slick Image Gallery? It didn’t use to be like that.
When I removed the component from my theme, both icons disappeared. When I added the component back, they both reappeared. Both buttons serve the same function.

[Chaboi_3000]

I can’t reproduce this using only the theme component.

[internet_cookie (Nick)]

콘솔에서 이 오류가 발생합니다:

`decorateCooked`는 메모리 누수를 방지하기 위해 `id` 옵션이 필요합니다

[mentalstring (Paulo)]

저희는 Slick과 Tile 이미지 갤러리를 모두 사용하고 있습니다(둘 다 훌륭합니다!). 한 사용자는 모바일 버전에서 Discourse 아이콘이 화면 밖으로 밀려나는 문제를 발견했습니다(예: 투표용 기어 아이콘 등):

이것은 composer 아이콘을 사용하는 Discourse 설치 환경에서 활성화된 다른 요소에 따라 발생하는 경계 사례(edge case)일 수 있다고 생각합니다. 어쨌든, 모바일에서 이미지 업로드 버튼은 실제로 오른쪽 하단에 위치해 있으며, 더 많은 아이콘을 배치할 여유 공간이 있는 영역입니다. 갤러리 아이콘도 그쪽으로 이동시키는 것이 합리적일까요? 그렇게 하면 상단 아이콘이 비좁게 배치되는 문제를 해결할 수 있을 뿐만 아니라, 논리적으로도 더 일관성이 생길 것입니다(모든 이미지 관련 아이콘을 한곳에 모으는 것).

[lyseoy]

배경색은 어디에서/어떻게 변경하나요?

아래 방법은 작동하지 않았습니다:

[serialcode (CodiHack)]

저도 같은 문제예요!!!

[Caswal_Parker]

이것을 설치했는데:

하지만 작성자 바(composer bar)에 나타나지 않아요?

[Caswal_Parker]

최근 업데이트로 인해 발생한 문제인데, 2.9.0 베타 4로 업데이트하면 해결된 것 같나요?