Skip to content

Group form elements and auto-expand ol attribution#120

Merged
emacgillavry merged 4 commits into
developfrom
accessibility-settings
Jun 30, 2021
Merged

Group form elements and auto-expand ol attribution#120
emacgillavry merged 4 commits into
developfrom
accessibility-settings

Conversation

@sdhoek
Copy link
Copy Markdown
Contributor

@sdhoek sdhoek commented Jun 29, 2021

No description provided.

@sdhoek sdhoek requested a review from emacgillavry June 29, 2021 14:53
var defaults = mergeConfig(CONFIG.BASE_DEFAULTS, basemaps.defaults);
if (!basemaps.layers || basemaps.layers.length < 0) {
err('no basemap defined, please define a basemap in the configuration');
console.log(basemaps);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please leave out the console.log.

Comment thread docs/app/views/index.twig
<span class="wizard-step-heading">Kies een library<span class="wizard-step-explanation">Let op de gebruiksvoorwaarden en prijzen. NLMaps / Kadaster is hiervoor niet verantwoordelijk.</span> </span>
</span>
<div class="wizard-step-content">
<li class="wizard-step" role="group" aria-labelledby="step-1-label">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Within each <li>, you can start a <fieldset>

@emacgillavry emacgillavry merged commit 8421709 into develop Jun 30, 2021
@emacgillavry emacgillavry deleted the accessibility-settings branch March 17, 2025 10:02
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