DennisSnijder / bolt-ui-options

Provides options with a simple UI
8 stars 3 forks source link

Trully allow theme OR extension file #14

Closed nlemoine closed 6 years ago

nlemoine commented 7 years ago

This fixes a behaviour that's a bit annoying.

I try to set some options in the theme.yml file. But I don't want to set any options in app/config/extensions/boltuioptions.snijder.yml.

This PR is a quick and dirty workaround addressing this bug.

I'm also trying to add some fields programmatically to provide a field that can be able to select dynamic content (a page, or any contenttype). Any plans to add such field type?

It's currently not very easy to achieve this (lacking public methods like addThemeTabs, addExtensionsTabs being protected).

But that's another topic. I'll come back to it later 😉

DennisSnijder commented 6 years ago

Woah! Never saw this PR! Good catch!