Skip to content

Switch details to Accordion in content#4724

Merged
frankharkins merged 4 commits intomainfrom
FH/accodions-migration-pt-2
Mar 5, 2026
Merged

Switch details to Accordion in content#4724
frankharkins merged 4 commits intomainfrom
FH/accodions-migration-pt-2

Conversation

@frankharkins
Copy link
Copy Markdown
Member

@frankharkins frankharkins commented Feb 26, 2026

This PR switches some <details> tags to the new <Accordion> component.

I haven't switched all the tags; some will require rewording to make the titles fit on one line. I only switched tags that didn't require any rewording.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@qiskit-bot
Copy link
Copy Markdown
Contributor

One or more of the following people are relevant to this code:

Comment thread docs/tutorials/index.mdx
Comment thread docs/tutorials/sample-based-quantum-diagonalization.ipynb
Comment thread learning/courses/quantum-safe-cryptography/cryptographic-hash-functions.ipynb Outdated
Comment thread learning/courses/quantum-safe-cryptography/cryptographic-hash-functions.ipynb Outdated
Copy link
Copy Markdown
Collaborator

@beckykd beckykd left a comment

Choose a reason for hiding this comment

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

This is going to be gorgeous! Thanks Frank!!

Comment thread docs/guides/execution-modes-faq.mdx Outdated
<summary>
Does Qiskit Runtime local testing mode support different execution modes?
</summary>
<span id="parallel"></span>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This ID belongs before the next accordion; not this one.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good catch, thanks for reviewing closely.

Comment thread learning/courses/quantum-business-foundations/quantum-technology.mdx Outdated
Comment thread learning/courses/quantum-diagonalization-algorithms/skqd.ipynb
frankharkins and others added 2 commits March 2, 2026 10:51
Co-authored-by: Rebecca Dimock <66339736+beckykd@users.noreply.github.com>
Co-authored-by: abbycross <across@us.ibm.com>
Comment thread learning/courses/quantum-safe-cryptography/quantum-safe-cryptography.ipynb Outdated
Comment thread docs/guides/function-template-chemistry-workflow.ipynb Outdated
Comment thread docs/guides/function-template-chemistry-workflow.ipynb Outdated
Comment thread docs/guides/install-qiskit.mdx Outdated
Comment thread docs/guides/metapackage-migration.mdx Outdated
Comment thread docs/guides/qiskit-addons-sqd-get-started.ipynb Outdated
Comment thread docs/guides/qiskit-addons-sqd.mdx Outdated
Copy link
Copy Markdown
Collaborator

@abbycross abbycross left a comment

Choose a reason for hiding this comment

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

I re-checked the code and only spot-checked the preview. Looks great

@frankharkins frankharkins added this pull request to the merge queue Mar 5, 2026
Merged via the queue into main with commit 4d2a944 Mar 5, 2026
6 checks passed
@frankharkins frankharkins deleted the FH/accodions-migration-pt-2 branch March 5, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants