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

[css-display] Missing definition of 'inline-list-item' #1495

Closed
Loirooriol opened this issue Jun 2, 2017 · 5 comments
Closed

[css-display] Missing definition of 'inline-list-item' #1495

Loirooriol opened this issue Jun 2, 2017 · 5 comments

Comments

@Loirooriol
Copy link
Contributor

Loirooriol commented Jun 2, 2017

The Precomposed Inline-level Display Values section defines the following <display-legacy> keywords:

  • inline-block: Behaves as inline flow-root.
  • inline-table: Behaves as inline table.
  • inline-flex: Behaves as inline flex.
  • inline-grid: Behaves as inline grid.

However, <display-legacy> allows a 5th possibility: inline-list-item.

According to the summary table, it should be

  • inline-list-item: Behaves as list-item inline flow.

Or, to maintain a similarity among all these definitions, the equivalent

  • inline-list-item: Behaves as inline list-item.
@Loirooriol
Copy link
Contributor Author

Or maybe inline-list-item should be dropped. All the other precomposed inline-level values are inline- followed by an inner display type, and exist for legacy reasons. But list-item is not an inner display type, and is a new addition.

@tabatkins tabatkins added the css-display-3 Current Work label Jun 5, 2017
@tabatkins
Copy link
Member

I don't think anyone has implemented inline-list-item yet - it's just specified over in Lists, as a useful feature. If so, we should just drop it, and let people write inline list-item if they want.

@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed Publishing an updated WD of css-align and css-display, and agreed to the following resolutions:

  • RESOLVED: Publish an updated WD of css-align
The full IRC log of that discussion <dael> Topic: Publishing an updated WD of css-align and css-display
<Rossen_> t- ?? wow :)
<dael> astearns: align it's an updated WD
<dael> fantasai: Yep. We wanted dbaron approval on that since a lot of the issues were filed by him. If he's okay publishing now or wants more time to look.
<dael> dbaron: I think there's more work to do, but I wouldn't obj to a new WD. I went through the commentor response required, well, all but 3. THere were 4 I marked as not satisfied.
<dael> fantasai: Great. publish an update and keep working
<ChrisL> that is an auto-publish, right?
<dael> astearns: Obj to updated WD of css align?
<fantasai> ChrisL, yes
<dael> RESOLVED: Publish an updated WD of css-align
<dael> astearns: Second is also updated WD of css-display. Looked like fewer issues.
<dael> fantasai: Fewer, but quite a few. We could do 1495 quickly before publish. We should do an update and then keep working.
<fantasai> https://github.com//issues/1495
<dael> github: https://github.com//issues/1495
<fantasai> GitHub: https://github.com//issues/1495

@astearns
Copy link
Member

astearns commented Jul 19, 2017

we confused the bot - the actual minutes on this issue are

fantasai: Suggestion was it's a little different then some of the others and do we really need this one. It might be worth dropping that unless someone has impl and wants to keep it.
astearns: Anyone know if inline-list-item has been imp?
Rossen_: I don't believe we are.
astearns: Obj to dropping inline-list-item?
fantasai: No hits in Mozilla's codebase
RESOLVED: Drop inline-list-item

@FremyCompany
Copy link
Contributor

We need to be kinder with the bot, or it might try to drown itself ;-)

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants