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.

Sign up for GitHub

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

Jump to bottom

Begin building the default UI in core or in image-sequencer-ui package #246

Closed
jywarren opened this issue May 4, 2018 · 6 comments
Closed

Begin building the default UI in core or in image-sequencer-ui package #246

jywarren opened this issue May 4, 2018 · 6 comments
Labels
help wanted more-details-please
Milestone
Core improvements

Comments

@jywarren
Copy link
Member

jywarren commented May 4, 2018

Take a look at #245 to see the new organization of the default HTML ui --

We should consider spinning that out into part of ImageSequencer core, or a new NPM package called image-sequencer-ui!

Connected to #232 (ui fixes) and #197
This was referenced May 4, 2018
Closed
Merged
@jywarren jywarren added this to the Default UI work milestone May 4, 2018
@jywarren
Copy link
Member Author

jywarren commented May 9, 2018

started with generic image selector in #253

@gitmate gitmate bot mentioned this issue Sep 28, 2018
Closed
@grvsachdeva
Copy link
Member

Hey @jywarren, please update this issue. Thanks!

@jywarren
Copy link
Member Author

I think we are good here, although we should open a new issue to move all UI files to /src/ui or /ui/src -- @publiclab/is-reviewers what do you think?

@harshkhandeparkar
Copy link
Member

Hi! I'm working on separating ui files in #825

@harshkhandeparkar
Copy link
Member

Will be addressed in a separate issue.

@harshkhandeparkar
Copy link
Member

#1238

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted more-details-please
Projects
None yet
Milestone
Core improvements
Development

No branches or pull requests
3 participants
@jywarren @grvsachdeva @harshkhandeparkar