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

Remove .info class name #10993

Merged
merged 1 commit into from
Mar 6, 2018
Merged

Remove .info class name #10993

merged 1 commit into from
Mar 6, 2018

Conversation

gormus
Copy link
Contributor

@gormus gormus commented Feb 28, 2018

The example suggest there is a .info class available for labels. However there is no definition for .info class in v6.x, the demo is simply rendering the default .label.

The example suggest there is a `.info` class available for labels. However there is no definition for `.info` class, the demo is simply rendering the default `.label`.
@DanielRuf
Copy link
Contributor

DanielRuf commented Feb 28, 2018

Right, it seems it was one of the deprecated settings and was still in the docs.

834be75#diff-03112641c36e40dca2d56b484a8e6079

https://foundation.zurb.com/sites/docs/global.html

Copy link
Contributor

@ncoden ncoden left a comment

Choose a reason for hiding this comment

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

👍 LGTM, I did not found other info in the code. Thanks @gormus

@ncoden ncoden merged commit b1e0cfd into foundation:master Mar 6, 2018
ncoden pushed a commit to ncoden/foundation-sites that referenced this pull request Jun 16, 2018
c016cf6 Remove `.info` class name

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>
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.

3 participants