Skip to content

Permit markup on prev/next session context buttons#3583

Merged
jcoyne merged 1 commit into
mainfrom
permit-markup-on-buttons
Apr 28, 2025
Merged

Permit markup on prev/next session context buttons#3583
jcoyne merged 1 commit into
mainfrom
permit-markup-on-buttons

Conversation

@jcoyne

@jcoyne jcoyne commented Apr 25, 2025

Copy link
Copy Markdown
Member

Previously, if a button was customized to have child DOM elements, such as:

<a href="..." data-context-href="..."><span>Text</span></a>

it would not receive the intended behavior. Becuase the click target would be a child element (the span in this case), not the element with the required dataset attributes

Previously, if a button was customized to have child DOM elements, such as:
```html
<a href="..." data-context-href="..."><span>Text</span></a>
```

it would not receive the intended behavior.  Becuase the click target would be a child element (the span in this case), not the element with the required dataset attributes
@jcoyne jcoyne merged commit 5f30abc into main Apr 28, 2025
@jcoyne jcoyne deleted the permit-markup-on-buttons branch April 28, 2025 14:35
jcoyne added a commit that referenced this pull request Apr 29, 2025
Previously, if a button was customized to have child DOM elements, such as:
```html
<a href="..."><span>Text</span></a>
```

it would not receive the intended behavior.  Becuase the click target would be a child element (the span in this case), not the link element.

This is similar to #3583
jcoyne added a commit that referenced this pull request Apr 29, 2025
Previously, if a button was customized to have child DOM elements, such as:
```html
<a href="..."><span>Text</span></a>
```

it would not receive the intended behavior.  Becuase the click target would be a child element (the span in this case), not the link element.

This is similar to #3583
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