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(Item): cleanup Item's floated example #1719

Merged
merged 1 commit into from
Jun 1, 2017

Conversation

qoalu
Copy link
Contributor

@qoalu qoalu commented May 31, 2017

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

Fixes error in docs. "AS" is not being recognized.
@qoalu qoalu changed the title docs(ItemExampleFloated): your description docs(ItemExampleFloated): Fix syntax error May 31, 2017
@qoalu qoalu changed the title docs(ItemExampleFloated): Fix syntax error docs(ItemExampleFloated): Fixes syntax error May 31, 2017
@codecov-io
Copy link

codecov-io commented May 31, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1719   +/-   ##
=======================================
  Coverage   99.75%   99.75%           
=======================================
  Files         142      142           
  Lines        2468     2468           
=======================================
  Hits         2462     2462           
  Misses          6        6
Impacted Files Coverage Δ
src/views/Item/ItemGroup.js 100% <0%> (ø) ⬆️

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 1a1b9e0...d0e45a3. Read the comment docs.

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.

As I see, example works correctly, however it's a cruft and we need to remove it.

@layershifter layershifter changed the title docs(ItemExampleFloated): Fixes syntax error docs(Item): cleanup Item's floated example May 31, 2017
@qoalu
Copy link
Contributor Author

qoalu commented May 31, 2017

The error appears after changing anything in the code. Here I changed a string value inside got this error:
https://www.dropbox.com/s/j4kuaq1wrq1us83/Screenshot%202017-05-31%2018.33.18.png?dl=0

Change any demo with ImageComponent in it here: https://react.semantic-ui.com/views/item

@levithomason
Copy link
Member

Yep this is a bug. The browser editor does some ugly regex for making sense of import statements in a browser. I did not account for as syntax there.

The initial render is OK because the example code editor text is not used. The first render is handled by the normal webpack'd app. On editor change, the editor code is parsed and compiled in-browser then re-rendered. Hence, the error happens on edit only.

@levithomason levithomason merged commit 34a37d7 into Semantic-Org:master Jun 1, 2017
layershifter pushed a commit to iamcaleberic/Semantic-UI-React that referenced this pull request Jun 1, 2017
Fixes error in docs. "AS" is not being recognized.
@levithomason
Copy link
Member

Released in semantic-ui-react@0.68.5.

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