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

Tracking Editor features 🧨 #627

Closed
14 tasks done
Sagarpreet opened this issue Oct 7, 2020 · 11 comments
Closed
14 tasks done

Tracking Editor features 🧨 #627

Sagarpreet opened this issue Oct 7, 2020 · 11 comments

Comments

@Sagarpreet
Copy link
Contributor

Sagarpreet commented Oct 7, 2020

Resolve outstanding issues remaining from the Summer 2020 Editor Team of 4 work, also documented here: https://pad.publiclab.org/p/editor-tasks

Editor repo tasks:

  • ensure the formatting toolbar (in both rich and markdown modes) doesn't stick to the bottom of the frame as we scroll far enough down or up that the text area is no longer shown at all
  • STATUS: Done
  • PR: fix toolbar stickness issue  #522

plots2 tasks:

  • comment
  • reply to comment
  • comment just posted (form inserted via AJAX)
  • reply to comment just posted
  • editing existing comments
  • editing comments just posted
  • editing comment replies
  • editing comment replies just posted
@welcome
Copy link

welcome bot commented Oct 7, 2020

Thanks for opening your first issue! This space is protected by our Code of Conduct - and we're here to help.
Please follow the issue template to help us help you 👍🎉😄
If you have screenshots or a gif to share demonstrating the issue, that's really helpful! 📸
Do join our Gitter channel for some brainstorming discussions.

@Sagarpreet
Copy link
Contributor Author

Done => Merged
Needs Clarity => Need help from @jywarren , @ebarry
Work in Progress => Something is left to do in the PR

@jywarren
Copy link
Member

This is looking awesome, Sagarpreet, thank you!

@ebarry
Copy link
Member

ebarry commented Oct 27, 2020

Thank you @Sagarpreet ! I don't see any more items labelled "needs clarification" -- is that right?

@Sagarpreet
Copy link
Contributor Author

Correct @ebarry 👍
These were resolved by Jeff and some by you on gitter 😄

@jywarren
Copy link
Member

I think we're just waiting on publiclab/plots2#8714 - publishign the Editor v3 to plots2 -- plus the Preview route and follow-ups, right?

@sagarpreet-chadha
Copy link
Contributor

Correct Jeff 👍

@Sagarpreet
Copy link
Contributor Author

NPM release for all above changes: #657
plots2 integration of all above changes: publiclab/plots2#8744

@jywarren
Copy link
Member

Noting bold bug PR at #659 as well! Thank you!!

@jywarren
Copy link
Member

Updated the Preview project checkboxes!!!! Yay!!!

@jywarren
Copy link
Member

This is all done, i believe!!! Thanks so much @Sagarpreet !!!!

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

4 participants