Skip to content

Commit

Permalink
CSS2 ED switched to Bikeshed, no longer multipage (#129)
Browse files Browse the repository at this point in the history
The Editor's Draft of CSS2 was converted to Bikeshed recently:
w3c/csswg-drafts#5196

This means that the spec becomes single page for now (at least until Bikeshed
supports multipage specs).

This update drops `cover.html` as "common" filename, since it was only used
there, and adjusts tests to only expect multiple page from the published
CSS 2.1 version.

Note the published version of CSS 2.2 still has multiple pages, so the spec
still needs to retain the `multipage` flag in `specs.json`.
  • Loading branch information
tidoust authored Jul 3, 2020
1 parent d17befb commit 0769511
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 10 deletions.
3 changes: 1 addition & 2 deletions src/determine-filename.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@ module.exports = async function (url) {
// Check common candidates
const candidates = [
"Overview.html",
"index.html",
"cover.html"
"index.html"
];

for (const candidate of candidates) {
Expand Down
6 changes: 0 additions & 6 deletions test/determine-filename.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,4 @@ describe("determine-filename module", function () {
const filename = await determineFilename(url);
assert.equal(filename, "Overview.html");
});

it("finds cover.html filenames", async () => {
const url = "https://drafts.csswg.org/css2/";
const filename = await determineFilename(url);
assert.equal(filename, "cover.html");
});
});
4 changes: 2 additions & 2 deletions test/extract-pages.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ describe("extract-pages module", function () {
assert.ok(pages.length > 20);
});

it("extracts pages from the CSS 2.2 spec", async () => {
const url = "https://drafts.csswg.org/css2/";
it("extracts pages from the CSS 2.1 spec", async () => {
const url = "https://www.w3.org/TR/CSS21/";
const pages = await extractPages(url);
assert.ok(pages.length > 20);
});
Expand Down

0 comments on commit 0769511

Please sign in to comment.