Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hugo: do not show anchor link in shortcode header #485

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anne-usmedia
Copy link
Collaborator

  • Moved css to only show the anchor hover state on direct descendents of article__content, so the
    anchor does not show inside a shortcode
  • Set behaviour of cursor so it does not show the pointer when we do not show the anchor

To test:

  • Go to /examples/shortcodes/info and check if the anchor icon does not show on hover on the
    shortcode titles
  • Also check other shortcodes for the same behaviour

Fixes: cue-lang/cue#3279

* Moved css to only show the anchor hover state on
direct descendents of article__content, so the
anchor does not show inside a shortcode
* Set behaviour of cursor so it does not show the
pointer when we do not show the anchor

To test:
* Go to /examples/shortcodes/info and check if
the anchor icon does not show on hover on the
shortcode titles
* Also check other shortcodes for the same
behaviour

Fixes: cue-lang/cue#3279

Signed-off-by: Anne van Gorkom <[email protected]>
Copy link

netlify bot commented Dec 18, 2024

Deploy Preview for cue ready!

Name Link
🔨 Latest commit f361ecd
🔍 Latest deploy log https://app.netlify.com/sites/cue/deploys/6762adb581066b0008361af0
😎 Deploy Preview https://deploy-preview-485--cue.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@jpluscplusm
Copy link
Collaborator

@anne-usmedia CI jobs should now work successfully on your PRs (i.e. go green, if everything's happy!) since we changed your PR branches to be in this repo and not a fork. The current failure on this PR appears to be real, as it's failing at npm run lint. Do let me know if you can't see the relevant log output and we'll fix that.

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.

cuelang.org: headings inside info blocks have a link icon on hover
2 participants