Skip to content

Conversation

webrdaniel
Copy link
Contributor

@webrdaniel webrdaniel commented Sep 30, 2025

Screenshot 2025-09-30 at 16 04 30

Note

Refactors LLM actions into a single dropdown menu and updates the doc header layout and styles, adding UI library/icon deps.

  • Frontend (theme):
    • LLM actions:
      • Replace CopyForLLM and ViewAsMarkdown buttons with a single dropdown in theme/LLMButtons/index.jsx using @apify/ui-library Menu and @apify/ui-icons.
      • Implements copy-as-Markdown (with loading/success feedback) and view-as-Markdown actions.
      • Removes theme/LLMButtons/CopyForLLM and theme/LLMButtons/ViewAsMarkdown components.
      • Revamps styles in theme/LLMButtons/styles.module.css for the new button and menu options.
    • Doc header layout:
      • In theme/DocItemContent/index.js, wraps title and LLM controls in a flex container (styles.docItemContent), keeping MDXContent below; adds styles.module.css for layout.
  • Dependencies:
    • Add @apify/ui-icons and bump @apify/ui-library; update lockfile accordingly.

Written by Cursor Bugbot for commit 5455d61. This will update automatically on new commits. Configure here.

@webrdaniel webrdaniel self-assigned this Sep 30, 2025
@github-actions github-actions bot added this to the 124th sprint - Web team milestone Sep 30, 2025
@github-actions github-actions bot added the t-web Issues with this label are in the ownership of the web team. label Sep 30, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment @cursor review or bugbot run to trigger another review on this PR

@apify-service-account
Copy link

Preview for this PR was built for commit 5455d61 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit e02ef01 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 9c3bcd0 and is ready at https://pr-1956.preview.docs.apify.com!

@birosrichard
Copy link
Contributor

I've added buttons that will "Open in ...

  • Claude
  • ChatGPT
  • Perplexity

I also need to add the icons to the ui-icons package. So we need to wait until the package is released to update the icons.
https://github.com/apify/apify-core/pull/23340

@webrdaniel webrdaniel requested a review from katzino October 1, 2025 10:07
Copy link
Contributor

@katzino katzino left a comment

Choose a reason for hiding this comment

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

💅

@jbartadev jbartadev requested review from patrikbraborec and removed request for birosrichard October 1, 2025 10:58
Copy link
Member

@jbartadev jbartadev left a comment

Choose a reason for hiding this comment

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

Looks good!

@hanatsai do we have the icons from ready? 🤔

CC @birosrichard

@apify-service-account
Copy link

Preview for this PR was built for commit bbf23417 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 2f130477 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 2ce622e0 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 8e3f17c3 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 7924c507 and is ready at https://pr-1956.preview.docs.apify.com!

@patrikbraborec
Copy link
Contributor

patrikbraborec commented Oct 1, 2025

Hi, one point. I think it now has duplicate buttons Copy for LLM and Copy page. I think that both buttons do the same thing, but have different titles. I think that instead of Copy page, we can just have View as Markdown and remove the Copy page button.

Also, would it be possible to add some kind of indication for the user that the page was copied. For example, change icon to "✅" or something like that?

@patrikbraborec
Copy link
Contributor

Also, the icons are little bit weird. Can we add logos of the tools instead of markdown icon, similar to coinbase docs:

image

@apify-service-account
Copy link

Preview for this PR was built for commit cfc4ba72 and is ready at https://pr-1956.preview.docs.apify.com!

@hanatsai
Copy link
Member

hanatsai commented Oct 1, 2025

I guessed we missed the "copied" behavior :) Figma

Screen.Recording.2025-10-01.at.15.33.12.mov

@apify-service-account
Copy link

Preview for this PR was built for commit c7b22b4d and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 3b3ca36a and is ready at https://pr-1956.preview.docs.apify.com!

@webrdaniel
Copy link
Contributor Author

webrdaniel commented Oct 1, 2025

@patrikbraborec @hanatsai The icons and copy state should be fixed. Regardless of the Copy page button, it is the same as the Coinbase version. I guess we will keep it as it is for now, and we can improve it in future iterations.

@jbartadev
Copy link
Member

We agreed with @patrikbraborec to rename the "Copy page" title in the dropdown to "Copy for LLM" just to keep it consistent with the main copy button (same way as Coinbase and Claude docs)

@apify-service-account
Copy link

Preview for this PR was built for commit 5b1acf8f and is ready at https://pr-1956.preview.docs.apify.com!

Copy link
Member

@hanatsai hanatsai left a comment

Choose a reason for hiding this comment

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

Sorry I'll pre-approve. I forgot the mobile version, can we fix it? 🙏 Figma

Screenshot 2025-10-01 at 16 02 41

@apify-service-account
Copy link

Preview for this PR was built for commit fbac4e80 and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 14fc012b and is ready at https://pr-1956.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 2c4180b9 and is ready at https://pr-1956.preview.docs.apify.com!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-web Issues with this label are in the ownership of the web team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants