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

docs(Introduction): fix declarative example #1704

Merged
merged 1 commit into from
May 25, 2017

Conversation

aabustamante
Copy link
Contributor

There is a difference between a code translated from JSX to HTML in https://react.semantic-ui.com/introduction where the max value is 5 in JSX version and 3 in HTML version

introduction declarative

This PR sets the max value in HTML version to 5

@aabustamante
Copy link
Contributor Author

oh, sorry. I don't know why my previous commits (for other pull requests) are here too.

@codecov-io
Copy link

codecov-io commented May 25, 2017

Codecov Report

Merging #1704 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1704   +/-   ##
=======================================
  Coverage   99.75%   99.75%           
=======================================
  Files         142      142           
  Lines        2468     2468           
=======================================
  Hits         2462     2462           
  Misses          6        6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d41235c...a5b4fb7. Read the comment docs.

@layershifter layershifter changed the title fix declarative example in introduction docs(Introduction): fix declarative example May 25, 2017
Copy link
Member

@layershifter layershifter left a comment

Choose a reason for hiding this comment

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

I've made a rebase to master, LGTM.

@levithomason levithomason merged commit 6bf47ab into Semantic-Org:master May 25, 2017
@levithomason
Copy link
Member

Released in semantic-ui-react@0.68.4.

aabustamante added a commit to aabustamante/Semantic-UI-React that referenced this pull request Jun 15, 2017
* docs(Button): remove redundant prop in Vertical Group example (Semantic-Org#1699)

* feat(typescript): Export generics types (Semantic-Org#1698)

* docs(Dropdown): fix world icon in search example (Semantic-Org#1695)

* Fix MenuExampleText

* add world icon to Dropdown example

* rename 'languageOptions' to 'languages'

* rename  languages   to languageOptions and separate line into two

* separate line of code in two

* separate props in different lines

* remove trailing spaces

* docs(Introduction): fix declarative example (Semantic-Org#1704)

* feat(Item): add unstackable prop to ItemGroup (Semantic-Org#1706)

* feat(Item): add unstackable prop to ItemGroup

* docs(Item): add example for unstackable

* chore(package): commit package-lock.json

* 0.68.4

* docs(changelog): update changelog [ci skip]

* docs(Icon): fix selector for input (Semantic-Org#1714)

* chore(package): update require-dir to version 0.3.2 (Semantic-Org#1721)

https://greenkeeper.io/

* docs(ItemExampleFloated): your description (Semantic-Org#1719)

Fixes error in docs. "AS" is not being recognized.

* chore(package): update react-ace to version 5.0.1 (Semantic-Org#1712)

https://greenkeeper.io/

* fix(Dropdown): add addition item key (Semantic-Org#1727)

* fix(factories): handle falsy `key` values (Semantic-Org#1729)

* fix(Dropdown): fix key handling

* fix(Dropdown): fix key handling

* fix(factories): handle falsy keys

* chore(package): update package-lock.json

* chore(package): update chai-enzyme to 0.7.1 (Semantic-Org#1731)

* feat(typings): expose FormComponent in typings (Semantic-Org#1680)

* Exposed form component in typings to support custom controls in packages.

* Update index.d.ts

* Update index.d.ts

* chore(package): update package-lock.json

* 0.68.5

* docs(changelog): update changelog [ci skip]

* fix(Input): add missing minLength prop (Semantic-Org#1734)

* docs(TableExampleSortable): pass in null when that column shouldn't be sorted (Semantic-Org#1737)

* feat(TextArea): add minHeight property, docs example (Semantic-Org#1679)

* add minHeight property, example to docs

* add rows prop/adjust minHeight prop usage

* mixed(TextArea): update docs, tests, props and typings

* fix(Textarea): move back minHeight prop to style

* fix(htmlInputProps): fix handle on falsy values (Semantic-Org#1746)

* fix(Search): Allow default action if there is no selected result (Semantic-Org#1742)

* docs(images): add missing images, update urls (Semantic-Org#1763)

* fix(Accordion): typings inverted to boolean (Semantic-Org#1758)

Tiny fix for typings, inverted type incorrectly set to string.

* feat(Icon): add ability use the loading prop without an icon (Semantic-Org#1768)

* feat(Button): add focus method (Semantic-Org#1764)

* fix(Dropdown): change active item on keyboard up/down (Semantic-Org#1735)

* fix(Dropdown): change active item on keyboard up/down

* fix(Dropdown): change active item on keyboard up/down

* refactor(Dropdown): simplify move constant

* refactor(Dropdown): remove hidden select (Semantic-Org#1730)

breaking(Dropdown): remove hidden select

* fix(Checkbox|Input): fix handling of aria-attributes (Semantic-Org#1752)

* fix(Checkbox|Input): fix handling of aria-attributes

* feat(htmlInputProps): update handling of aria
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants