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-content][css-gcpm] Duplicate property definition for string-set #6435

Open
tidoust opened this issue Jul 8, 2021 · 13 comments
Open

[css-content][css-gcpm] Duplicate property definition for string-set #6435

tidoust opened this issue Jul 8, 2021 · 13 comments

Comments

@tidoust
Copy link
Member

tidoust commented Jul 8, 2021

The string-set property (and companion string() and content() functions are defined twice:

I suspect the definition in css-content is the "right" one. Could the definition in css-gcpm be dropped?

Via w3c/webref#127 (comment)

@cdoublev
Copy link
Collaborator

I do not know if CSS Content has "authority" over CSS GCPM but the definition from CSS Content is compatible with the definition from CSS GCPM but not the other way around, ie. a valid value parsed against the definition from CSS Content will always be successfully parsed against the definition from CSS GCPM.

@tabatkins
Copy link
Member

We should probably just retire GCPM to Note status, I guess? It has no active editors and no planned changes to that status, plus no web-facing implementations of the remaining bits of it.

@faceless2
Copy link

We should probably just retire GCPM to Note status

Please don't do this. Not yet.

We're all aware that GCPM has issues but it's formed the basis for a number of independent implementations and has been largely unchanged for a decade. There is at least some compatibility across implementations as a result, and this has been the case for many years.

There's cleanup to be done, certainly: in order to implement page number counters you need to consult Content-3, Lists-3 , Pages-3 but not GCPM-3. Bookmarks were moved to Content-3, despite not generating any content on the page while Page selectors are in GCPM despite having nothing to do with content, and should probably be in Page-3. It needs more and better testcases (cough web-platform-tests/wpt#21701) and it needs an editor.

So I would also like to have a discussion on this, but with the goal of fixing it, not retiring it. If an editor is required for this and the WG would consider me for it I'd certainly like to know more about what's involved. I'll make sure I'm on the next call.

What I don't want to do is make a case specifically against retiring it, rather than for improving it, on only a few days notice in the comments on an unrelated issue where the people that will be affected most are unlikely to see it.

@fantasai
Copy link
Collaborator

Side comment: the latest Process now has a dedicated status for discontinued REC-track work. See https://www.w3.org/2021/Process-20211102/#abandon-draft

@faceless2
Copy link

There are eight different CSS layout engines that I know of that implement at least some of this specification, with various degrees of interop, and it now has a new editor. So although GCPM might not have had a lot of love recently, it's no longer abandoned. I'm hoping to put some time into it over the next few months, once I've clarified the process.

@svgeesus
Copy link
Contributor

Strongly in favor of cleanup and re-evaluation rather than quietly and per-emptively discontinuing without asking any of the actual implementers.

@faceless2 if you need any help as editor, let me know.

@tabatkins
Copy link
Member

Now that it has an editor, I'm fine with it staying around and getting fixed. My "let's just retire it" was based on it being abandoned and left in a bad state.

@svgeesus
Copy link
Contributor

So when counting independent implementations, please don't exclude candidates based on their target medium, especially when the specification targets that exact medium.

This used to be easier: Shepherd had a way to manually upload test results from non-browser implementations, although each test had to be run by importing it, one by one, looking at the PDF or whatever output it used, and then jotting down in a file the pass or failure.

We now have on WPT automated running of all tests - but in four browsers, three of which use different engines, and with no way to inject non-browser test results. And no way to get results for tests that require a human to judge the pass/fail (so gradients are entirely un-tested, for example).

@tabatkins
Copy link
Member

rather than quietly and per-emptively discontinuing without asking any of the actual implementers.

Also y'all are really giving me lots of power I don't actually have; this would have been done with WG discussion, not by me just summarily changing the status and pushing a new publication on my own. Thus the wording from my post, "we should probably [retire it]"; it was a suggestion, not me banishing it from existence.

@svgeesus
Copy link
Contributor

My faith in your necromantic powers is severely shaken

@faceless2
Copy link

Thank you both and yes, I will be asking questions!

@bernhardf-ro
Copy link

[repost of 2h old post from correct account]

I'd like to add support to @faceless2 stating that there are in fact implementations of the GCPM and other specifications targeting paged output. That they are not web-facing is not unexpected for specifications that target print and PDFs. So when counting independent implementations, please don't exclude candidates based on their target medium, especially when the specification targets that exact medium.

Regarding the GCPM specifically, I think that it should preferably only be retired after all remaining parts have been moved to different specifications. There is not much left, and all of it should fit into existing specifications. But if nobody is working on it, it's probably better to make that known.

@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed Duplicate property definition for string-set.

The full IRC log of that discussion <TabAtkins> Topic: Duplicate property definition for string-set
<TabAtkins> github: https://github.com//issues/6435
<TabAtkins> jfkthame: Issue filed on spec I now own
<astearns> s/jfkthame/faceless
<TabAtkins> faceless: Need to review specs taht use <content-list>
<TabAtkins> faceless: Will fix once review
<TabAtkins> chris: Think reason is that GCPM's definition wasn't speific enough, so we just forked it. Should just align them.
<TabAtkins> faceless: I think we do *need* two slightly differnet definitions
<TabAtkins> faceless: Will fix when I'm back form vacation
<TabAtkins> astearns: Propose we wait until you evaluate it, then
<TabAtkins> astearns: Is the dupe causing immediate problems?
<TabAtkins> chris: Causing a problem with automated validators
<TabAtkins> TabAtkins: w3c tooling is complaining about two different definitions. They can maintain a manual fix for now but would like it to be correct in source
<chris> q+

@astearns astearns removed the Agenda+ label Jul 27, 2022
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

9 participants