Skip to content

Add detailed explanation of the JS components configuration merging process#37303

Merged
XhmikosR merged 2 commits intomainfrom
gs/popover-fix-doc
Mar 27, 2023
Merged

Add detailed explanation of the JS components configuration merging process#37303
XhmikosR merged 2 commits intomainfrom
gs/popover-fix-doc

Conversation

@GeoSot
Copy link
Copy Markdown
Member

@GeoSot GeoSot commented Oct 12, 2022

Description

Add some detailed explanation about the confg merging process of our JS components, and fix misleading popover & tooltip options description

Checklist

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • My change introduces changes to the documentation
  • I have updated the documentation accordingly

Live previews

Related issues

closes #37298

  • It needs proofreading

@GeoSot GeoSot marked this pull request as draft October 12, 2022 12:03
@GeoSot GeoSot requested review from mdo and patrickhlauke October 12, 2022 12:03
@GeoSot GeoSot force-pushed the gs/popover-fix-doc branch from cfde8ec to 0c1af65 Compare October 26, 2022 21:50
@GeoSot GeoSot marked this pull request as ready for review October 26, 2022 21:55
@GeoSot GeoSot force-pushed the gs/popover-fix-doc branch from 0c1af65 to ecd1968 Compare October 26, 2022 21:55
Copy link
Copy Markdown
Member

@mdo mdo left a comment

Choose a reason for hiding this comment

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

Took a quick pass at editing the copy here. No need for the <small>. I'm not 100% on what we're saying here though, so wouldn't mind eyes from @XhmikosR or @julien-deramond.

Comment thread site/layouts/partials/js-data-attributes.md Outdated
@GeoSot GeoSot force-pushed the gs/popover-fix-doc branch from 8168c4c to e9d1e38 Compare November 9, 2022 00:08
@GeoSot GeoSot force-pushed the gs/popover-fix-doc branch from 175acb3 to 4cea03f Compare November 17, 2022 20:13
Comment thread site/layouts/partials/js-data-attributes.md Outdated
@XhmikosR XhmikosR force-pushed the gs/popover-fix-doc branch from 7e7bae1 to 3a5e438 Compare March 27, 2023 09:29
@XhmikosR XhmikosR merged commit 94c7dc7 into main Mar 27, 2023
@XhmikosR XhmikosR deleted the gs/popover-fix-doc branch March 27, 2023 09:30
@mahilanmjd mahilanmjd mentioned this pull request Apr 16, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

Popover JS config object overrides data-bs- attributes

4 participants