Skip to content

WIP: bootstrap popovers - #2

Open
stephenmcm wants to merge 12 commits into
symbiote-library:masterfrom
stephenmcm:feature-bootstrap
Open

WIP: bootstrap popovers#2
stephenmcm wants to merge 12 commits into
symbiote-library:masterfrom
stephenmcm:feature-bootstrap

Conversation

@stephenmcm

Copy link
Copy Markdown

No description provided.

@nglasl

nglasl commented Jan 16, 2017

Copy link
Copy Markdown

Commenting here for reference, can you please look into adding a toggle so it's possible to switch between the existing jQuery UI and Bootstrap?

Comment thread templates/InlineHelp.ss
@@ -1,44 +1,17 @@
<% if HelpItems %>

@nglasl nglasl Jan 17, 2017

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If the current page doesn't have inline help items, you end up getting JS errors while trying to loop over an undefined variable. Can you please do something such as (outside the if statement)..

var SS_InlineHelpItems = {};

Or check whether this actually exists in the JS.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Heh good point. Fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants