Skip to content

shortcode helper map height, using transients for geocoder - #273

Closed
hupe13 wants to merge 2 commits into
bozdoz:masterfrom
hupe13:master
Closed

shortcode helper map height, using transients for geocoder#273
hupe13 wants to merge 2 commits into
bozdoz:masterfrom
hupe13:master

Conversation

@hupe13

@hupe13 hupe13 commented Jun 10, 2025

Copy link
Copy Markdown
Collaborator

Hi Bozdoz,

I changed the shortcode helper page because reported bug.
I learned how to use transients. I have now applied this in the geocoder.

Please review.

@bozdoz

bozdoz commented Jun 10, 2025

Copy link
Copy Markdown
Owner

Thanks! I'll take a look

@bozdoz

bozdoz commented Jun 11, 2025

Copy link
Copy Markdown
Owner

I'd prefer to leave the transients in this pr:

#217

#217

That pr stalled obviously as the original reporters stopped reviewing. I think switching options to transients is a breaking change. I realize it's ideal, but I wanted to plan out a good migration workflow. Again, the interest in the idea seems to have died off. Anyway, I think we leave transients for that pr/issue. If you don't mind removing those from this one.

Comment thread templates/shortcode-helper.php Outdated
$replace = array(
'leaflet-map height=250 ',
'leaflet-image height=250 ',
'leaflet-wms height=250 ',

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might not be the best way to do this, but I guess it works. I would have thought we could set height via css and make the css "!important".

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have thought we could set height via css and make the css "!important".

Thank you for hint.

@hupe13

hupe13 commented Jun 12, 2025

Copy link
Copy Markdown
Collaborator Author

If you don't mind removing those from this one.

I remove this. I'm sorry, I didn't take #217 into account.

@sonarqubecloud

Copy link
Copy Markdown

@hupe13 hupe13 closed this Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants