Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Merging into jsPsych repo? #2

Open
jodeleeuw opened this issue Apr 20, 2016 · 1 comment
Open

Merging into jsPsych repo? #2

jodeleeuw opened this issue Apr 20, 2016 · 1 comment

Comments

@jodeleeuw
Copy link

Hi,

This looks like a really useful set of features being developed on this fork, and it seems like it targets at least one of the major open issues in the jsPsych repository: jspsych/jsPsych#244.

Are you interested in merging into the main repository at some point? Or is this a separate fork at this point?

Best,
Josh

@garrickb
Copy link
Collaborator

garrickb commented Apr 20, 2016

Most of our styling is going to be done using the Angular Material library, which I think is a very specialized use-case for us, and involves changing which HTML elements jsPsych injects into the DOM (for example, it uses <md-progress-linear> for a progress bar).

I can see how having SASS in the jsPsych library would be useful, and that would require a change on how development is done for jsPsych by requiring something like gulp to compile the SASS files into CSS. The only change we've done on that in this repo is condensing the styling a bit into a nested structure in the SASS file.

With that being said, I don't think the work we've done here would carry over very well to the main jsPsych repo, since we have made changes in here mainly for our own special uses.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants