Entering more than one clause along with and modal is not shown on the main page.
How can I make the modal appear in the whole web?
Greetings and thanks
Related issues:
- #3076322: Add a instruction message in the "pages" field
- #3076075: Check if modal is published before to show
- #3075825: Add method getModalToShow
- #3075020: Create method getCurrentPath
- #3074979: Create method getAuthenticatedUserName
- #3073920: Create method clearText
- #3073642: Add method getModalToShowByPage in ModalPage
- #3073640: Add method getModalToShowByParameter in ModalPage
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | modal_page-Option_to_create_a_Modal_to_Show_in_All_Pages-3066013-2-D8.patch | 1.05 KB | thalles |
Comments
Comment #2
renatog commentedHello @igonzalez, how are you?!
Thanks for reporting, but I can't reproduce this error on my machine.
Please, can you put here a step-by-step to reproduce this error?
I'll work on the fix for us.
Best,
Comment #3
renatog commentedIf you create 2 modals and configure both to show on front page
<front>only the second modal will be shown.Check the example bellow:
Configuration:

Result:

The current rule is to show the last modal
What's your idea / suggestion ?
Comment #4
renatog commentedComment #5
igonzalez commentedI'm sorry, I've worked it out.
I would like to know how I can make it look on all the web pages
Comment #6
renatog commentedReally makes sense.
This option there is no yet, but it's a good Feature Request.
We can work on that
Comment #7
renatog commentedComment #8
renatog commentedSuggestion to it:

We can leave this field blank to show the modal in all pages.
What do you think, @thalles?
@igonzalez thanks for your suggestion.
P.s. All patches are welcome
Good Week.
Best,
Comment #9
renatog commentedComment #10
thallesThe suggestion is really good!
But, I think the best solution would be one code like us , because a field blank is for inactive modal.
What you think @RenatoG?
Comment #11
thallesComment #12
thallesHello @RenatoG and @igonzalez!
Follow the patch, that add the new feature, can you test for me?
Comment #13
thallesComment #14
thallesComment #15
renatog commentedReally works good!
Go ahead Thalles!
Keep pushing, dude
Comment #16
renatog commentedAfter that, we can create a new release for this.
What do you think?
Comment #18
thalles