janboddez / indieblocks

Use blocks, and, optionally, "short-form" post types to more easily "IndieWebify" your WordPress site.
https://indieblocks.xyz
GNU General Public License v3.0
22 stars 3 forks source link

Improve Location block #88

Closed janboddez closed 1 year ago

janboddez commented 1 year ago

This should probably be a dynamic block, so that it can be used in a theme template.

janboddez commented 1 year ago

If we had a separate Weather block, we could add an option for icons and/or a description. Wouldn’t have to worry about a "separator" as admins could just add whatever in between the two blocks (if used together).

janboddez commented 1 year ago

Wouldn’t have to worry about a "separator" as admins could just add whatever in between the two blocks (if used together).

But there'd be no way to conditionally hide this separator, it'd be hardcoded into the template; so if the weather API is down or the plugin misconfigured, we'd get a separator but no weather info (for instance).

Better keep 'em together and provide a text field or something, let users choose a separator character.

janboddez commented 1 year ago

A text field, or a ... ButtonGroup RadioGroup? Choose from a number of predefined charaters? (We could always do a backend filter, too.)