Tiles Image Gallery

theme-component

(Joe) #1

Responsive Grid Galleries!

10KB (Gzip)
3 http requests on initial load
IE 11+ & modern browsers

Github repository link

https://github.com/hnb-ku/Discourse-Tiles-image-gallery




Built using David Desandro’s open source Masonry and ImagesLoaded libraries - MIT license

" … released under the MIT license. Have at it."


How do I install this?

You install this component just like any other theme. Follow the instructions in the official guide:

Once you’re done, simply add the component to your current theme. Done!


How do I use this?

This component adds a button to the composer.

So all you have to do is:

1- Upload a bunch of images using the default Discourse upload button then highlight the code:

2- Click the gallery button.

You will get a static preview of the the selected images in the preview pane:

Done!


Anything else I need to know?

This component does not have any color / background styles in it. It will inherit the current theme’s styles.

Here’s the theme component running on a different theme:

This theme component also maintains the default Discourse lighbox behavior, this means that clicking any of the images will open them in the native lightbox viewer.

Can I use both Slick and Tiles at the same time?

You better believe it! :smile:

This is possible by making both slick and tiles components of a theme.

Like this:

You can then make “my theme” in the example above the default theme or make it user-selectable and it would include both Slick and Tiles.


A big thank you to everyone that helped along the way: :yum:

@Dax - Composer button
@cpradio - Button translations
@Mittineague - ajaxComplete handler
@Simon_Cossar - api.onPageChange handler
@vinothkannans - Topic page url regex
@sam - adding data-theme-* to whitelisted attributes


Bug reports / Suggestions / PRs welcome. :wine_glass:


DEPRECATED: Discourse masonry gallery theme component
Image Gallery Plugin
Formatting toolbar
#2

You demonstrated that both Slick & Tiles theme can be used at the same time, but did not explain how. Could you please? (explain how)


(Joe) #3

That’s a very good point @kaefert thank you for bringing it up :sunflower:
I will edit both the Slick and Tiles topics and clarify what I meant.

In the meantime, to answer your question:

Tile and Slick act independently. What I meant to say here:

is that there is no conflict between Tiles and Slick. If you want to use both Tiles and Slick at the same time, you can! You just have to install both of them, separately.

If you install both of the same time, you get something like this in the composer:

Capture

And if you use both of them in a topic/post you end up with something like this:


#4

The problem I have about activating both at the same time is that I only can set one as default in the admin settings, and can only select one of them in the user settings.
Screenshot%20from%202018-03-10%2015-33-32_crop


(Joe) #5

I understand :ok_hand:

What you want you do is make Slick and Tiles components of your current theme instead of them being user-selectable themes.

Like this:

You can then make “my theme” in the example above the default theme or make it user-selectable and it would include both Slick and Tiles.


(Bartusevicius Mindaugas) #6

Bugs*


(Joe) #7

Thanks for reporting @savas

Can you share a little bit more details? The reason why I’m asking is because that looks like a onebox. This theme component only applies styles to images within very specific structure like so

.tiles-initialized {
 // all styles are applied here
}

So it should not have any effect on oneboxes at all.

Does the issue still occur if you disable the the theme temporarily?


(Bartusevicius Mindaugas) #8

i got installed facebook onebox plugin can he mess your plugin ?


(Joe) #9

1- The Facebook onebox plugin is not supported anymore, you don’t need it.
2- No, there should not be any conflict between the two. Tiles only targets specific images but you have to use the button in the composer to make it work

image

If you’re not sure, check the OP again and let me know if you need more help.


#10

Both the Slick theme component and this one don’t seem to work after upgrading discourse to 2.0.0.beta5. JS Error messages for this one:

I18n.translations.en.js.composer is undefined
@https://discourse-instance/:104:2
O@https://discourse-instance/assets/application-d85c1a58a506702e3c09017c9d92a23100e1ec3c2a620a10440601587bf2d226.js:13:23523
initialize@https://discourse-instance/assets/application-d85c1a58a506702e3c09017c9d92a23100e1ec3c2a620a10440601587bf2d226.js:6:8870
runInstanceInitializers/<@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:23844
n</e.prototype.each@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:8774
n</e.prototype.topsort@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:8272
t</e.prototype.topsort@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:7670
_runInitializer@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:24023
runInstanceInitializers@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:23786
_bootSync@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:11232
didBecomeReady@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:8:17098
invokeWithOnError@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:7:19387
flush@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:7:19832
flush@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:7:20592
end@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:7:20937
run@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:7:22006
join@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:7:22126
u.join@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:13:3164
u.bind/<@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:13:3370
p@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:2:9019
o/</f<@https://discourse-instance/assets/ember_jquery-27e777857b8c0730dacfe09cb11711365d21a5db4f9ee0b85d494e4259cf6cda.js:2:9324

(Joe) #11

Hi @kaefert, @vchrizz and thank you for reporting :sunflower:

My apologies for not getting to you earlier.

the good news is that I have just tested on v2.0.0.beta5 +120 and both Slick and Tiles seem to be working.

My guess is that subsequent commits to Discourse addressed the issue.

Are you still facing the same problem?

Even better news is that I tinkered a little bit with both Slick and Tiles and they now have much more targeted firing methods that will only fire when needed. Credit here goes to @vinothkannans for the awesome tip.

So if you’re using either Slick or Tiles or both, please update now and let me know if the problem is still there, or if there are any other issues that need to be addressed.


Slick Image Gallery
#12

I just upgraded our Discourse instance to the latest commit and reinstalled the two themes components. Everything works again as it worked with beta4! :slight_smile:

Thanks for your help!


(Lars) #13

I am facing a Problem on beta 2.0.7.
I imported the Theme and configured it as a Child-Theme for my standard “Dark”-Theme.

2018-04-25%2012_01_21-Administration%20-%20Kneipentruppe%20-%20Forum


(Lars) #14

(Had to split the post because I cannot include more than one image in my posts)

However I just get a blank Screen…any ideas on that?


(Joe) #15

@Lars Can you check if there are any errors in the browser console?


(Lars) #16


(Joe) #17

Ok, thank you. Can you update the theme and check again?


(Lars) #18

Did an Update and refreshed (cleared cache).
Seems to be the same error.


(Joe) #19

Ok, try it again one last time :sunflower:. If that does not work I’ll take a deeper look at this and fix it within the next couple of hours.


After the fact edit

Crosspost:


(Lars) #20

That did the trick, thank you very much for the quick help :slight_smile: