Skip to content

Conversation

@dhruvshr
Copy link

Description

This PR fixes the inconsistent italic styling between single-line and multi-line comments in the tokenColorCustomizations settings within the README file where it talks about disabling italics. When I tried that, it only disabled the italics for the single-line comments, and not the multi-line comments (both double-quoted and single-quoted). PS. I was working in python3.12 when I noticed this.

Context

I'm new to open source contributions and would appreciate any feedback on:

  • Whether this is the right approach to fix this issue
  • If the documentation changes are clear and helpful
  • Any other improvements I could make

Changes

  • Added string.quoted.docstring.multi pattern
  • Added Python-specific docstring patterns
  • Added support for both single and double quoted multi-line comments

Thank you for taking the time to review my contribution!

Fix inconsistent italic styling between single-line and multi-line comments
in the tokenColorCustomizations settings. Previously, the italic disabling only
worked for single-line comments, leaving multi-line comments and Python docstrings
in italics. This change should ensure consistent non-italic styling across all comment types.

Changes include:
- Add string.quoted.docstring.multi pattern
- Add Python-specific docstring patterns
- Support both single and double quoted multi-line comments
"string.quoted.docstring.multi",
"string.quoted.docstring.multi.python punctuation.definition.string.begin",
"string.quoted.docstring.multi.python punctuation.definition.string.end",
"string.quoted.docstring.multi.python constant.character.escape"
Copy link

Choose a reason for hiding this comment

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

ig you can make the same thing with bold, quote, etc.

but this is a perfect way to do italic font 👍

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.

2 participants