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.

Sign up for GitHub

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

Jump to bottom

Feature/update toggles jquery ui #112

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Feature/update toggles jquery ui #112

wants to merge 11 commits into from

Commits on Mar 9, 2017

  1. Merge pull request #1 from andrew-worsfold/master 

    Updating fork to master 3/9/2017
    @kipraske
    kipraske committed Mar 9, 2017
    Configuration menu
    Copy the full SHA
    f129a80 View commit details
    Browse the repository at this point in the history

  2. Including jQueryUI library with only tabs and accordions

    @kipwustl
    kipwustl committed Mar 9, 2017
    Configuration menu
    Copy the full SHA
    e2395e1 View commit details
    Browse the repository at this point in the history

  3. Oh yeah, jquery-ui is built into wordpress. Okay. Let's try it

    @kipwustl
    kipwustl committed Mar 9, 2017
    Configuration menu
    Copy the full SHA
    2e56897 View commit details
    Browse the repository at this point in the history

  4. Oh actually toggles = accordions. 

    Alright this isn't quite working as we expect. We need a HTML structure
of...

```

accordion-wrapper
 h3
 div
 h3
 div
 h3
 div

```

Here is what we have

```

accordion-wrapper
 toggle-wrapper
  h3
  div
 toggle-wrapper
  h3
  div

```
    @kipwustl
    kipwustl committed Mar 9, 2017
    Configuration menu
    Copy the full SHA
    e576df1 View commit details
    Browse the repository at this point in the history

Commits on Mar 10, 2017

  1. Successfully hooked up toggles jQuery UI for toggles 

    We just needed to define the header option for the toggles
    @kipwustl
    kipwustl committed Mar 10, 2017
    Configuration menu
    Copy the full SHA
    4ff6fa1 View commit details
    Browse the repository at this point in the history

  2. Added script from stackoverflow to implement "accordion" button behavior

    @kipwustl
    kipwustl committed Mar 10, 2017
    Configuration menu
    Copy the full SHA
    61edb4a View commit details
    Browse the repository at this point in the history

  3. Removing original code and wp-style-guidifying the rest for toggles

    @kipwustl
    kipwustl committed Mar 10, 2017
    Configuration menu
    Copy the full SHA
    eaf5777 View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2017

  1. Missing Link: Event listeners on toggles working again 

    Turns out I needed to use onInitialize or the built in event listeners
would totally not work. In order to get the backend canvas working we
need those events like onChildChange etc to reinitialize the accordion,
and to get those events we needed to go though the right channels
    @kipwustl
    kipwustl committed Apr 10, 2017
    Configuration menu
    Copy the full SHA
    774dcec View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2017

  1. Merge branch 'master' of https://github.com/kipraske/tailor

    @kipwustl
    kipwustl committed Apr 12, 2017
    Configuration menu
    Copy the full SHA
    c21404f View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into feature/update-toggles-jquery-ui

    @kipwustl
    kipwustl committed Apr 12, 2017
    Configuration menu
    Copy the full SHA
    7855212 View commit details
    Browse the repository at this point in the history

  3. Don't need the jquery-ui-tabs module if i couldn't get it to work

    @kipwustl
    kipwustl committed Apr 12, 2017
    Configuration menu
    Copy the full SHA
    f7b85a2 View commit details
    Browse the repository at this point in the history