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

Revise sidebarwidgets.rst to add details on Largo widgets, and fix errors #1160

Merged
merged 15 commits into from
Mar 17, 2016

Conversation

jackbrighton
Copy link

The sidebarwidgets page on http://largo.readthedocs.org/users/sidebarswidgets.html was woefully incomplete and out of date, and revising it has been a high priority prior to the coming release of Largo 0.5.5. (see: #847)

Legacy language and formatting was cleaned up considerably and normalized. Sections were added for Largo widgets that weren't in the docs, and other sections were revised for accuracy, spelling, language, etc. Formatting was applied for better usability. Basically this is a complete revamp of the page.

It was suggested to create pages for each widget, so as to provide more details, screenshots, etc., and that can certainly be done. Since it was a high priority to fill in missing information and to correct errors, I chose to first make these changes to the existing page.

A readable version of the revised page can be found here: https://github.com/inn/docs/blob/master/how-to-work-with-us/pull-requests.md

Jack Brighton and others added 2 commits March 4, 2016 18:21
aschweigert added a commit that referenced this pull request Mar 17, 2016
Revise sidebarwidgets.rst to add details on Largo widgets, and fix errors
@aschweigert aschweigert merged commit 4e1b37b into master Mar 17, 2016
@aschweigert aschweigert deleted the jack-docs-sidebarwidgets branch March 17, 2016 14:45
@aschweigert
Copy link

A couple small issues (will make this a separate ticket, but noting here for reference):

  • if we add the link roundups widget we should also add the saved links widget (added by the same plugin
  • this should actually probably be a separate section including widgets added by commonly used plugins (link roundups, dfp, analytic bridge, etc.)
  • we need to add a section re: the default WP widgets that we de-register (and maybe explain why), the list can be found in inc/widgets.php
  • the taxonomy list widget will need updated docs when Revisit the Taxonomy List widget #1171 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants