Skip to content

Change view type to a radio so that only one type is selectable (D888) - #645

Open
droberts-ctrlo wants to merge 1 commit into
ctrlo:devfrom
droberts-ctrlo:view-type-toggle
Open

Change view type to a radio so that only one type is selectable (D888)#645
droberts-ctrlo wants to merge 1 commit into
ctrlo:devfrom
droberts-ctrlo:view-type-toggle

Conversation

@droberts-ctrlo

Copy link
Copy Markdown
Contributor

No description provided.

@droberts-ctrlo droberts-ctrlo changed the title Change view type to a radio so that only one type is selectable Change view type to a radio so that only one type is selectable (D888) Aug 19, 2026

@pwlodarski-ctrlo pwlodarski-ctrlo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

PR reviewed - appears fine, just one minor clarification comment.

const value = this.el.data("radio-value");

if (!target || !value) {
console.error("RadioRevealComponent: Missing data-radio-target or data-radio-value attribute.");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is this picked up by our /scripterror frontend error catching endpoint?

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