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

Gh pages audit #680

Merged
merged 27 commits into from
Oct 2, 2014
Merged

Gh pages audit #680

merged 27 commits into from
Oct 2, 2014

Conversation

cmcculloh-kr
Copy link

This audit covers the overview and usage section for each component. The examples sections will come next (there are plans for gist driven examples with jsfiddle type tinker-boxes). Lots of work to be done yet here, but, I think this is a good stopping point for the "first pass".

</p>
</div>

<p>If not present on the page onready, these controls will be 'lazily' initialized when the user interacts with them (such as on <code>mouseover</code>). <span class="text-warn">Be careful of <code>changed</code> events when initializing. To disable this data-api, use <code>$.off('fu.data-api')</code></span>.</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

" this section's" and "These controls" seem ambiguous. Due to placement it makes me think it's the controls in "Requires Manual Initialization"

Copy link
Author

Choose a reason for hiding this comment

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

Fixed. Thanks @interactivellama

@kevinparkerson kevinparkerson added this to the 3.1.0 milestone Sep 29, 2014
@futuremint futuremint self-assigned this Sep 30, 2014
<ul>
<li>All plugins depend on jQuery <i>and</i> Bootstrap; these must be loaded <strong>before</strong> Fuel UX.</li>
<li>Version requirements for jQuery/Bootstrap are documented in our <a href="{{ site.repo }}/blob/{{ site.current_version }}/bower.json"><code>bower.json</code></a> file.</li>
<li>Check dependencies if including controls individually <span class="text-muted">(console errors will indicate unmet dependencies). Some controls depend on others.</span></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

This paragraph is more helpful than the previous one. It also clarifies that these are all jQuery plugins, whereas the previous docs assumed the reader could figure this out on their own which is not good technical writing.

Copy link
Contributor

Choose a reason for hiding this comment

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

Minor note, should be "Fuel UX", not "FuelUX" (eg Fuel UX is a collection of jQuery plugins.)


<h5 id="pillbox-examples-sample-methods">Sample Methods</h5>
<div class="btn-group">
<button class="btn btn-default" id="btnPillboxAdd">add</button>
Copy link
Contributor

Choose a reason for hiding this comment

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

None of these buttons appear to work.

Copy link
Contributor

Choose a reason for hiding this comment

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

The pillbox sample method buttons are working in live gh pages.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed.

@futuremint
Copy link
Contributor

:shipit:

futuremint added a commit that referenced this pull request Oct 2, 2014
@futuremint futuremint merged commit 4072180 into gh-pages Oct 2, 2014
@interactivellama interactivellama deleted the gh-pages-audit branch October 9, 2014 20:11
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.

6 participants