Skip to content

Conversation

amolenaar
Copy link
Member

@amolenaar amolenaar commented Jan 9, 2024

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the current behavior?

[subject] should check for the existance of the subject property, not its value.

Issue Number: #2994, #3023

What is the new behavior?

The [subject] expression now checks for the presence of the subject attribute, whether it's set or not.

Does this PR introduce a breaking change?

  • Yes
  • No

Fix CSS attribute.

Other information

@github-actions github-actions bot added the python Pull requests that update Python code label Jan 9, 2024
@amolenaar amolenaar requested a review from danyeaw January 9, 2024 19:42
@amolenaar amolenaar linked an issue Jan 9, 2024 that may be closed by this pull request
2 tasks
`[subject]` should check for the existance of the subject property,
not its value.
Copy link
Member

@danyeaw danyeaw left a comment

Choose a reason for hiding this comment

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

Nice fix, thanks @amolenaar!

@danyeaw danyeaw added fix A fix for a bug and removed python Pull requests that update Python code labels Jan 10, 2024
@danyeaw danyeaw merged commit 7ffb376 into main Jan 10, 2024
@danyeaw danyeaw deleted the fix-css-attribute branch January 10, 2024 02:54
@danyeaw danyeaw changed the title Fix attribute check Fix CSS attribute rules for subject properties Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix A fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSS "highlight disconnect associations" causes all items to be red
2 participants