Skip to content

Rename .tt-dropdown-menu to .tt-menu#24

Closed
j0hj0h wants to merge 1 commit intohyspace:masterfrom
j0hj0h:master
Closed

Rename .tt-dropdown-menu to .tt-menu#24
j0hj0h wants to merge 1 commit intohyspace:masterfrom
j0hj0h:master

Conversation

@j0hj0h
Copy link

@j0hj0h j0hj0h commented May 9, 2015

Rename .tt-dropdown-menu to .tt-menu for compatibility with typeahead.js v0.11.1, like mentioned in #22

Rename .tt-dropdown-menu to .tt-menu for compatibility with typeahead.js v0.11.1
@j0hj0h
Copy link
Author

j0hj0h commented May 9, 2015

Overlooked, that there are tests to adjust.

@j0hj0h j0hj0h closed this May 9, 2015
@dlong500
Copy link

Is this resolved now? I don't see these changes merged yet. If there are still tests to adjust why is the issue closed?

@rtyley
Copy link

rtyley commented Jun 20, 2015

Looks like this is a more comprehensive version of the earlier pull request #23 , but it hasn't been merged because there are still tests to fix.

Until this is merged, a workaround is just to tell typeahead.js to use the old class name:

$('#elemId .typeahead').typeahead({
  classNames: {
    menu: 'tt-dropdown-menu'
  }
}

rtyley added a commit to rtyley/submitgit that referenced this pull request Jun 21, 2015
Note the workaround for `typeahead.js-bootstrap3.less` not having quite
updated to cope with a class-name change in `typeahead.js` v0.11.1 (made
the suggestions background-transparent) - just tell typeahead.js to use
the old class name:

hyspace/typeahead.js-bootstrap3.less#24 (comment)
rtyley added a commit to rtyley/submitgit that referenced this pull request Jun 27, 2015
Note the workaround for `typeahead.js-bootstrap3.less` not having quite
updated to cope with a class-name change in `typeahead.js` v0.11.1 (made
the suggestions background-transparent) - just tell typeahead.js to use
the old class name:

hyspace/typeahead.js-bootstrap3.less#24 (comment)
rtyley added a commit to rtyley/submitgit that referenced this pull request Jun 30, 2015
Typeahead for identifying message-ids in a human-readable way

Particularly like linking back to the Gmane article...

lotsa jquery

Add bootstrap-3-specific styling to the typeahead.js stuff

Note the workaround for `typeahead.js-bootstrap3.less` not having quite
updated to cope with a class-name change in `typeahead.js` v0.11.1 (made
the suggestions background-transparent) - just tell typeahead.js to use
the old class name:

hyspace/typeahead.js-bootstrap3.less#24 (comment)
rtyley added a commit to rtyley/submitgit that referenced this pull request Jun 30, 2015
Typeahead for identifying message-ids in a human-readable way

Particularly like linking back to the Gmane article...

lotsa jquery

Add bootstrap-3-specific styling to the typeahead.js stuff

Note the workaround for `typeahead.js-bootstrap3.less` not having quite
updated to cope with a class-name change in `typeahead.js` v0.11.1 (made
the suggestions background-transparent) - just tell typeahead.js to use
the old class name:

hyspace/typeahead.js-bootstrap3.less#24 (comment)
rtyley added a commit to rtyley/submitgit that referenced this pull request Jun 30, 2015
Using typeahead.js for confirming the message-id - we call into the
Gmane & MARC mail archives to show a summary of the message to the user.

Added bootstrap-3-specific styling to the typeahead.js stuff. Note the
workaround for `typeahead.js-bootstrap3.less` not having quite updated to
cope with a class-name change in `typeahead.js` v0.11.1 (made the
suggestions background-transparent) - just tell typeahead.js to use
the old class name:

hyspace/typeahead.js-bootstrap3.less#24 (comment)

lotsa jquery, coding like it's 2010.
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.

3 participants