[css-transitions-1] transition-property and transition shorthand conflict on whether none can be listed #2031

fantasai · 2017-12-03T00:56:08Z

transition-property: none | <single-transition-property>#
transition: [ [ none | <single-transition-property> ] || <time> || <single-timing-function> || <time> ]#

The transition shorthand allows none to be listed multiple times, but the transition-property shorthand it's supposed to be assigned into doesn't allow that. One of these syntaxes is therefore wrong.

The text was updated successfully, but these errors were encountered:

Parsing tests for animation and transition shorthands. https://drafts.csswg.org/css-animations/#animation https://drafts.csswg.org/css-transitions/#transition-shorthand-property Open issues: w3c/csswg-drafts#2031 w3c/csswg-drafts#4034 w3c/csswg-drafts#4223

… shorthand, a=testonly Automatic update from web-platform-tests [css-animations] [css-transitions] parse shorthand (#19195) Parsing tests for animation and transition shorthands. https://drafts.csswg.org/css-animations/#animation https://drafts.csswg.org/css-transitions/#transition-shorthand-property Open issues: w3c/csswg-drafts#2031 w3c/csswg-drafts#4034 w3c/csswg-drafts#4223 -- wpt-commits: edffcf9ef316c6906591e734cea5bb42c8a60fdf wpt-pr: 19195

WPT for animation and transition shorthand computed values. Open spec issues: w3c/csswg-drafts#2031 w3c/csswg-drafts#4034 w3c/csswg-drafts#4223 Change-Id: I3cac663f8a4cdce53856a9152e7b38033be9a76b

WPT for animation and transition shorthand computed values. Open spec issues: w3c/csswg-drafts#2031 w3c/csswg-drafts#4034 w3c/csswg-drafts#4223 Change-Id: I3cac663f8a4cdce53856a9152e7b38033be9a76b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1827317 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#700825}

…nd transition, a=testonly Automatic update from web-platform-tests CSS: Test computed value for animation and transition WPT for animation and transition shorthand computed values. Open spec issues: w3c/csswg-drafts#2031 w3c/csswg-drafts#4034 w3c/csswg-drafts#4223 Change-Id: I3cac663f8a4cdce53856a9152e7b38033be9a76b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1827317 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#700825} -- wpt-commits: 8530693c047e221fb2f55070082eb79620c56084 wpt-pr: 19320

…nd transition, a=testonly Automatic update from web-platform-tests CSS: Test computed value for animation and transition WPT for animation and transition shorthand computed values. Open spec issues: w3c/csswg-drafts#2031 w3c/csswg-drafts#4034 w3c/csswg-drafts#4223 Change-Id: I3cac663f8a4cdce53856a9152e7b38033be9a76b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1827317 Auto-Submit: Eric Willigers <ericwilligerschromium.org> Commit-Queue: Stephen McGruer <smcgruerchromium.org> Reviewed-by: Stephen McGruer <smcgruerchromium.org> Cr-Commit-Position: refs/heads/master{#700825} -- wpt-commits: 8530693c047e221fb2f55070082eb79620c56084 wpt-pr: 19320 UltraBlame original commit: c7015155594cd0b9aca8ec471c3ae4119fc62fec

cdoublev · 2022-09-08T11:21:30Z

Did you missed the transition prose for this rule? (explained in this thread from the archive)

If there is more than one <single-transition> in the shorthand, and any of the transitions has none as the <single-transition-property>, then the declaration is invalid.

EDIT: actually this rule does not make sense because <single-transition-property> does not include none.

Or do you mean that this rule should be hard coded in its value definition, eg.:

Name: transition
Value: <single-transition> [ , [<single-transition-property> || <time> || <easing-function> || <time>]# ]?

<single-transition> = <'transition-property'> || <time> || <easing-function> || <time>

Obviously it would be less confusing to rename <single-transition-property>.

fantasai added the css-transitions-1 Current Work label Dec 3, 2017

ewilligers mentioned this issue Sep 21, 2019

[css-animations] [css-transitions] parse shorthand web-platform-tests/wpt#19195

Merged

chromium-wpt-export-bot mentioned this issue Sep 26, 2019

CSS: Test computed value for animation and transition web-platform-tests/wpt#19320

Merged

cdoublev mentioned this issue Oct 17, 2022

[scroll-animations-1] scroll-timeline and view-timeline shorthand syntax #7627

Closed

cdoublev mentioned this issue Feb 10, 2023

[css-animations-1] Restrict none in animation and animation-name #8440

Closed

cdoublev mentioned this issue Mar 14, 2023

[css-backgrounds-4] Allow repeating none in box-shadow-offset #8567

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[css-transitions-1] transition-property and transition shorthand conflict on whether none can be listed #2031

[css-transitions-1] transition-property and transition shorthand conflict on whether none can be listed #2031

fantasai commented Dec 3, 2017

cdoublev commented Sep 8, 2022 •

edited

Loading

[css-transitions-1] transition-property and transition shorthand conflict on whether none can be listed #2031

[css-transitions-1] transition-property and transition shorthand conflict on whether none can be listed #2031

Comments

fantasai commented Dec 3, 2017

cdoublev commented Sep 8, 2022 • edited Loading

cdoublev commented Sep 8, 2022 •

edited

Loading