Skip to content

make it so that mathplayer message only appears when mathjax is on the p... - #1082

Merged
adampalay merged 1 commit into
masterfrom
adam/accessible-mathjax
Sep 20, 2013
Merged

make it so that mathplayer message only appears when mathjax is on the p...#1082
adampalay merged 1 commit into
masterfrom
adam/accessible-mathjax

Conversation

@adampalay

Copy link
Copy Markdown
Contributor

...age

Couple fixups to https://github.com/edx/edx-platform/pull/1002

  1. I made a mistake with the syntax on the IE conditionals

  2. I realized that I only wanted the mathplayer browser message to appear when the browser wasn't compatible AND there was mathjax on the page.

Thanks @talbs and @singingwolfboy

@singingwolfboy

Copy link
Copy Markdown
Contributor

Seems reasonable to me. 👍

@talbs

talbs commented Sep 20, 2013

Copy link
Copy Markdown
Contributor

Looks good from here, too. 👍

adampalay added a commit that referenced this pull request Sep 20, 2013
make it so that mathplayer message only appears when mathjax is on the p...
@adampalay
adampalay merged commit 6805f4b into master Sep 20, 2013
@adampalay
adampalay deleted the adam/accessible-mathjax branch September 20, 2013 20:51
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Oct 20, 2016
afzaledx pushed a commit to open-craft/openedx-platform that referenced this pull request Jun 1, 2018
MCKIN-3983: diagnosticfeedback version bump to v0.2.4
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