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

Demo page update #164

Closed
wants to merge 5 commits into from
Closed

Conversation

clubajax
Copy link
Contributor

Made minor change to the demo HTML page to persist the tab selection across refreshes. This is a critical feature since this page doubles as the unit tests. On small screens all the button clicking gets a bit painful.

Also tweaked the tabs CSS to be more responsive an not collapse.

# Conflicts:
#	README.md
#	dist/pptxgen.bundle.js
#	dist/pptxgen.bundle.js.map
#	dist/pptxgen.js
#	examples/pptxgenjs-demo.js
# Conflicts:
#	README.md
#	dist/pptxgen.bundle.js
#	dist/pptxgen.bundle.js.map
#	dist/pptxgen.js
@gitbrent
Copy link
Owner

gitbrent commented Sep 3, 2017

Hi @clubajax ,

The library needs to retain IE11 compatibility for now, so can you use a non-flex layout?

@clubajax
Copy link
Contributor Author

clubajax commented Sep 3, 2017

@gitbrent gitbrent self-requested a review September 9, 2017 02:04
@gitbrent gitbrent self-assigned this Sep 9, 2017
@gitbrent gitbrent added this to the 1.8.0 milestone Sep 9, 2017
gitbrent pushed a commit that referenced this pull request Sep 10, 2017
@gitbrent
Copy link
Owner

Thanks @clubajax !

I ended-up siwtching to Flex on the individual tab fieldsets as well, so the whole page display nicely on any size screen.

@gitbrent gitbrent closed this Sep 10, 2017
gitbrent pushed a commit that referenced this pull request Sep 10, 2017
@clubajax clubajax deleted the demo-page-update branch September 23, 2017 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants