Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Adds BLM banner #2454

Closed
wants to merge 2 commits into from
Closed

Adds BLM banner #2454

wants to merge 2 commits into from

Conversation

claudiopro
Copy link
Contributor

@claudiopro claudiopro commented Jun 2, 2020

Summary

Adds a Black Lives Matter banner like other OSS projects of the React family

Test Plan

Local tests

cd website
yarn 
yarn start

Screenshots

Screen Shot 2020-06-02 at 21 10 00

Screen Shot 2020-06-02 at 21 09 18

Screen Shot 2020-06-02 at 21 10 07

@claudiopro
Copy link
Contributor Author

claudiopro commented Jun 2, 2020

@yangshun I have no idea how to add a banner above the navbar. Is this possible with a declarative config like the one we have here?

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@claudiopro has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@claudiopro
Copy link
Contributor Author

Thank you @kedromelon!

@facebook-github-bot
Copy link

@claudiopro merged this pull request in 2a0c5bb.

@yangshun
Copy link
Contributor

yangshun commented Jun 3, 2020

@claudiopro the current PR works, but if you want to apply it to all pages, you can use the announcement bar feature.

An example PR - facebookexperimental/Recoil#241

@claudiopro
Copy link
Contributor Author

Thank you @yangshun, I'll look into this today!

@RAYDENFilipp
Copy link

Because of this change, the site became inaccessible

What happened to links
image

It is hard to see links.

vilemj-Viclick pushed a commit to kontent-ai/draft-js that referenced this pull request Jul 16, 2020
Summary:
**Summary**

Adds a Black Lives Matter banner like other OSS projects of the React family

**Test Plan**

Local tests

```
cd website
yarn
yarn start
```

## Screenshots
<img width="752" alt="Screen Shot 2020-06-02 at 21 10 00" src="https://user-images.githubusercontent.com/860099/83565342-2a8d7a80-a516-11ea-9eb8-697f1f781a4c.png">

<img width="1392" alt="Screen Shot 2020-06-02 at 21 09 18" src="https://user-images.githubusercontent.com/860099/83565335-26615d00-a516-11ea-8e6f-cad0d3f3cc2f.png">

<img width="1072" alt="Screen Shot 2020-06-02 at 21 10 07" src="https://user-images.githubusercontent.com/860099/83565349-2bbea780-a516-11ea-85d8-c9da292998c7.png">
Pull Request resolved: facebookarchive#2454

Reviewed By: mrkev

Differential Revision: D21849517

Pulled By: claudiopro

fbshipit-source-id: c4b8944b7cbfbe9603bde4ad5128f3826ecb1f04
alicayan008 pushed a commit to alicayan008/draft-js that referenced this pull request Jul 4, 2023
Summary:
**Summary**

Adds a Black Lives Matter banner like other OSS projects of the React family

**Test Plan**

Local tests

```
cd website
yarn
yarn start
```

## Screenshots
<img width="752" alt="Screen Shot 2020-06-02 at 21 10 00" src="https://user-images.githubusercontent.com/860099/83565342-2a8d7a80-a516-11ea-9eb8-697f1f781a4c.png">

<img width="1392" alt="Screen Shot 2020-06-02 at 21 09 18" src="https://user-images.githubusercontent.com/860099/83565335-26615d00-a516-11ea-8e6f-cad0d3f3cc2f.png">

<img width="1072" alt="Screen Shot 2020-06-02 at 21 10 07" src="https://user-images.githubusercontent.com/860099/83565349-2bbea780-a516-11ea-85d8-c9da292998c7.png">
Pull Request resolved: facebookarchive/draft-js#2454

Reviewed By: mrkev

Differential Revision: D21849517

Pulled By: claudiopro

fbshipit-source-id: c4b8944b7cbfbe9603bde4ad5128f3826ecb1f04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants