Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

IGNORE prefers-color-scheme article for web.dev #929

Closed
wants to merge 107 commits into from

Conversation

tomayac
Copy link
Member

@tomayac tomayac commented May 23, 2019

This pull request adds a new:

  • Guide(s) 📘
  • Codelab(s) 🔬

Have you read the wiki on writing a guide or a codelab?

  • Done! 🍰(but adhering is WIP)

Does your content tell a user how to fix a Lighthouse audit?
Have you added the Lighthouse audit ID to your YAML?

  • My changes do not mention Lighthouse audits. 🙅‍♂️
  • I have added the Lighthouse audit IDs to my content. 👍

Does your content reference any Glitch projects?
Have you read the wiki on working with Glitch?
Have you added the Glitch project to our team page?

  • My changes do not contain any Glitches. 🙅‍♂️
  • I have read the wiki on working with Glitch. 🍰
  • I have added my Glitches to the team page. 👏

mihajlija and others added 14 commits May 15, 2019 18:54
@tomayac tomayac added the DO NOT MERGE Actively working on but experimental label May 23, 2019
@tomayac tomayac added the blog label May 23, 2019
robdodson and others added 3 commits May 23, 2019 10:16
* Adds guidance about error boundaries

* Update src/site/content/en/react/code-splitting-suspense/index.md

Co-Authored-By: Houssein Djirdeh <[email protected]>

* Update src/site/content/en/react/code-splitting-suspense/index.md

Co-Authored-By: Houssein Djirdeh <[email protected]>

* Add jeffposnick
Copy link
Contributor

@robdodson robdodson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Started adding comments and then noticed this was a draft PR.
Give me a shout when you're ready for a review :)

date: 2019-05-23
hero: macosx.png
alt: macOS X Dark Mode preferences.
description: |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need a description so it'll appear in search.

alt: macOS X Dark Mode preferences.
description: |

tags:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
tags:
tags:
- post

post tag is required to show up on the blog landing page. I need to figure out a better way to make sure all blog posts get it automatically :\

@@ -0,0 +1,174 @@
---
title: Hello Darkness, My Old Friend—Dark Theme is Here
subhead:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need a catchy subhead. This will appear on the post's card on the blog landing page and under the title on the article itself.

@robdodson robdodson removed the request for review from Meggin May 23, 2019 18:19
@netlify
Copy link

netlify bot commented Jun 18, 2019

Deploy preview for web-dev-staging ready!

Built with commit 8ad2cf5

https://deploy-preview-929--web-dev-staging.netlify.com

@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of all the commit author(s), set the cla label to yes (if enabled on your project), and then merge this pull request when appropriate.

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: no Contributor has not signed the CLA and removed cla: yes Contributor has signed the CLA labels Jun 18, 2019
@tomayac tomayac marked this pull request as ready for review June 19, 2019 14:55
@tomayac tomayac requested a review from robdodson June 19, 2019 14:55
@tomayac
Copy link
Member Author

tomayac commented Jun 20, 2019

Sorry, I realized somehow this PR contains completely unrelated changes. Let me reopen.

@tomayac tomayac closed this Jun 20, 2019
@tomayac tomayac deleted the prefers-color-scheme branch June 20, 2019 06:39
@tomayac
Copy link
Member Author

tomayac commented Jun 20, 2019

☝️ Started fresh in #1021.

@tomayac tomayac changed the title prefers-color-scheme article for web.dev IGNORE prefers-color-scheme article for web.dev Jun 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
beta related cla: no Contributor has not signed the CLA DO NOT MERGE Actively working on but experimental
Projects
None yet
Development

Successfully merging this pull request may close these issues.