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

jspsych-form styles and jquery dependencies #374

Closed
hugueslacroix opened this issue Apr 12, 2017 · 5 comments
Closed

jspsych-form styles and jquery dependencies #374

hugueslacroix opened this issue Apr 12, 2017 · 5 comments
Milestone

Comments

@hugueslacroix
Copy link

I started using jspsych-form plugin and I find it very useful, however, it looks like there hasn’t been a lot of testing on it since the merge.
So far, I noticed a few issues:

  • It doesn’t show properly if you load both the MDL CSS and jsPsych/css/jspsych.css. It seems the issue comes from .jspsych-display-element, .jspsych-content-wrapper and .jspsych-content styles.
  • There are still a few lines with jQuery dependency:
  • There’s an issue when you preload something since there’s nothing in the plugin to update the MDL element when they are appended dynamically after the initial loading of the page. We could add this in the Form.prototype.render to solve the problem :
// instead of setTimeout I should probably use a listener that I don’t know yet, any recommendation?  
    setTimeout(function(){
       componentHandler.upgradeDom();
     }, 1);

Should I try and fix it? What should I know about the future CSS development in jsPsych?

#310

@jodeleeuw
Copy link
Member

You are absolutely right that there has not been a lot of testing on it yet. If you are interested in fixing it, that'd be great. The goal is to make it compatible with the base jspsych.css file, remove jQuery dependencies, and fix any other outstanding issues.

My goal for future CSS development is to get the flexbox stuff working consistently across browsers, and have all the basic elements minimally styled to a consistent simple look.

@hugueslacroix
Copy link
Author

Ok, I'll try to fix and update the plugin. Removing jQuery dependencies should not be a big issue, it looks like most of the work have already been done.

About the CSS, I'll try to understand how flexbox works and make the plugin work with the base jspsych.css, but it might take me a little while since I'm not that familiar with the flexbox.

@jodeleeuw jodeleeuw added this to the 6.0 milestone May 19, 2017
@jodeleeuw
Copy link
Member

@GavinQ1 can you take a look at this when you have some free time? I'd love to get this and #135 resolved.

@hugueslacroix
Copy link
Author

I have been working quite a lot on jspsych-form plugin in my own branch, I changed and improved many things, but I'm not ready yet to send you a pull request (I have a short deadline to analyze my data now). If this can wait a little more, I'll offer a solution, however, if @GavinQ1 can have a look at the bug it might be solved more quickly.

@GavinQ1
Copy link
Contributor

GavinQ1 commented May 26, 2017

got it!

jodeleeuw added a commit that referenced this issue Jun 19, 2017
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

No branches or pull requests

3 participants