Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

fix(stories): avoid to use linear-background it breaks Argos every time #804

Merged
merged 1 commit into from
Jan 5, 2018

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Jan 5, 2018

Summary

Use an other style attribute for the custom root prop, because the background with the linear-gradient breaks Argos almost every time.

@samouss samouss requested a review from Haroenv January 5, 2018 09:33
@@ -41,7 +41,7 @@ stories
Root: 'div',
props: {
style: {
background: 'linear-gradient(80deg, #00D8FF, #00A7FF)',
padding: '30px',
Copy link
Contributor

Choose a reason for hiding this comment

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

what about: border: 1em solid red? that will still be very easy to see, but will also show up in Argos

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Great idea!

@algobot
Copy link
Contributor

algobot commented Jan 5, 2018

Deploy preview ready!

Built with commit 0ba5708

https://deploy-preview-804--react-instantsearch.netlify.com

@samouss samouss force-pushed the fix/stories-custom-root branch from ddbdb35 to 0ba5708 Compare January 5, 2018 09:46
@samouss samouss merged commit 0beded7 into master Jan 5, 2018
@samouss samouss deleted the fix/stories-custom-root branch January 5, 2018 10:51
samouss added a commit that referenced this pull request Jan 9, 2018
<a name="4.4.1"></a>
## [4.4.1](v4.4.0...v4.4.1) (2018-01-09)

### Bug Fixes

* **SearchBox**: clear SearchBox without search as you type ([#802](#802)) ([c49b2b6](c49b2b6))
* **connectRange:** check if facet exist before access ([#797](#797)) ([6520760](6520760))
* **stories:** avoid to use linear-background it breaks Argos every time ([#804](#804)) ([0beded7](0beded7))
* **stories:** limit hits per page on Index ([#806](#806)) ([6eb14d3](6eb14d3))

### Features

* **Index:** allow custom root ([#792](#792)) ([d793b0a](d793b0a))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants