Masafumi Koba

Results 705 comments of Masafumi Koba

@romainmenke Just to make sure, you think this issue is a bug of `no-invalid-double-slash-comments` rather than a documentation issue, right?

Okay, let's address both in this issue. I'm okay with separating pull requests. 👍🏼

@Mouvedia @romainmenke Thanks for addressing this issue. https://github.com/stylelint/stylelint/issues/7904#issuecomment-2275024780 sounds good to me. You can directly open a PR for bug fixes or open a new issue if you need to....

@Mouvedia @romainmenke Since all the subtasks are resolved, can we close this issue?

@Mouvedia Thanks for the report. Supporting `transition-behavior` sounds good to me. Just to make sure, we can confirm this bug for `declaration-block-no-redundant-longhand-properties` on [this demo](https://stylelint.io/demo/#N4Igxg9gJgpiBcICGACYAdAdugLjgTkpgM4CWOpEmAtAA74S0z44Ce8KSANlwNxa4CRMhSrUoAV0KjMHAAwA6AEwBWYv2x5CJcpRoUAtqUwBzagDMJmMDI4wkxGBsHaRe8TC5J2KOeoFawrpiAEYwABZIAG6U+BzcXBAA7uKkxGD4MDhOWAC+IAA0IOakXDAAckgGcIgwAB5VtGUKYMTEheBUJSYIIBiYKCjoIPXZmFDEwxwA2liDg8PEbGVcxjjUkJjd1EtEUEj4UMNzKAC6eR2b3QBiEPgGSDi9AFbEVB2wtO2I-QsgS6wVmspkMQF5skthgUTotlp41hsuqQzLtxgcjghQeCYJCQBdckA): ```css a { transition-property: all;...

Additionally, [this demo](https://stylelint.io/demo/#N4Igxg9gJgpiBcICGACYAdAdugLjgTkpgM4CWOpEmAtAEYwAWSAbpfvCkgDZcQDu1KKWJh8MHDADcWXASJkKVDty4oADADoATAFZiKGEmIx1xaZgC+IADQgAZqS4wAckgC2cRDAAe7gA5OGmDExDbgVA4A5gggGJgoKOggPhKYUMRJHADaWAkJScQ4AJ5OXKSYONSQmFHUhURQSPhQSbkoALpYVrbVUQBiEPhuSDgxAFbEVGGwfqGIcfkghSUwZRWZiSBcIzCFSdZtBcWl5ZW9pJF1OA1NLQib2xJ7IF0gFkA) shows a *false negative* bug for `declaration-block-no-shorthand-property-overrides`: ```css a { transition-behavior: allow-discrete; /* ❌ It is not reported now, but it should be reported. */ transition: all...

@Mouvedia What reason do you think so? IMO, it seems more reasonable to change the existing logic `declaration-block-no-redundant-longhand-properties` than adding `ignoreLonghands` to `stylelint-config-standard`: https://github.com/stylelint/stylelint/blob/ee3118d2460cddf469f959b5438d8aeab7e9584e/lib/rules/declaration-block-no-redundant-longhand-properties/index.mjs#L116-L117 See also: ```js 'declaration-block-no-redundant-longhand-properties': true, ```...

I see that you are concerned about "the low browser support". But even if `declaration-block-no-redundant-longhand-properties` and `declaration-block-no-shorthand-property-overrides` support `transition-behavior`, the backward compatibility will be kept, right?

@gapkukb Thanks for the report. I confirm the behavior in [this demo](https://stylelint.io/demo/#N4Igxg9gJgpiBcIBG0CeAdAdgAm5ANhAE7ZExRa4BmEmALgLQDuMAlgOYAWd2AjAAz8QAGhBVW+GADkAhgFs4iGAA95AB0kA6AM51U+EeFrj2CEMErZ0IFXRiYo26-GwBtS7mu79MfK3oM3vgArtoA9LoyDjJEFCIeViBBvv6MkJgmDGRgMNraDMSwRNaWALpYAL6G6SYAYsRyMnRmAFbatIawatpmFjiJyX70zom8AGyaAOya-NbCll56kkNpxhxZMDl5BbEwxQiJAKwzmrxzC0lLKQFBoSPWvCezIJUgFUA). ``` Expected "font-weight" to come before "color" (order/properties-order) ``` But I'm not sure if this insertion of an...