Comments

igonzalez created an issue. See original summary.

renatog’s picture

Status: Active » Postponed (maintainer needs more info)

Hello @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,

renatog’s picture

Issue summary: View changes
StatusFileSize
new75.52 KB
new78.06 KB

If 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 ?

renatog’s picture

igonzalez’s picture

I'm sorry, I've worked it out.
I would like to know how I can make it look on all the web pages

renatog’s picture

Really makes sense.

This option there is no yet, but it's a good Feature Request.

We can work on that

renatog’s picture

Title: Dont show in front » Option to create a Modal to Show in All Pages
Category: Bug report » Feature request
Priority: Major » Normal
Status: Postponed (maintainer needs more info) » Active
renatog’s picture

StatusFileSize
new95.22 KB

Suggestion 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,

renatog’s picture

thalles’s picture

The 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?

thalles’s picture

Status: Active » Needs work
thalles’s picture

Status: Needs work » Needs review
StatusFileSize
new1.05 KB

Hello @RenatoG and @igonzalez!
Follow the patch, that add the new feature, can you test for me?

thalles’s picture

Issue summary: View changes
thalles’s picture

Version: 8.x-2.4 » 8.x-2.x-dev
renatog’s picture

Status: Needs review » Reviewed & tested by the community

Really works good!

Go ahead Thalles!

Keep pushing, dude

renatog’s picture

After that, we can create a new release for this.

What do you think?

  • thalles authored 3a72acd on 8.x-2.x
    Issue #3066013 by thalles, RenatoG, igonzalez: Option to create a Modal...
thalles’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.