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

PoC Highlighting code changes in code blocks #529

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mribeirodantas
Copy link
Member

No description provided.

Copy link

netlify bot commented Feb 28, 2025

Deploy Preview for nextflow-training ready!

Name Link
🔨 Latest commit 9dd4bac
🔍 Latest deploy log https://app.netlify.com/sites/nextflow-training/deploys/67ce36e21fe5870008a6448d
😎 Deploy Preview https://deploy-preview-529--nextflow-training.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.

@vdauwera
Copy link
Collaborator

Oh I see, yeah that looks really nice, we should do that consistently across all the courses. Something to do in a separate pass I think. Fwiw I didn't systematically do the before/after thing in the RNAseq materials to save time but I really like the look of it. Let's plan to do a campaign to add that in everywhere.

@ewels
Copy link
Member

ewels commented Mar 5, 2025

Yeahhh - I was thinking about exactly this as I went through the material 💯

I was wondering if by adding the highlighted lines we could get rid of the "before" / "after" and have only the "after" blocks 🤔 - or perhaps have as a grouped code block with "after" shown by default? Not sure if that's going too far though.

Agree that we should do it systematically, but I don't think it'd take that long to do.

@vdauwera vdauwera added documentation Improvements or additions to documentation improve New feature or request hello series labels Mar 5, 2025
@ewels ewels marked this pull request as draft March 8, 2025 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation hello series improve New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants