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

[Planning] Standardize Editors + Remaining bugs #648

Open
1 of 10 tasks
sashadev-sky opened this issue Oct 27, 2020 · 7 comments
Open
1 of 10 tasks

[Planning] Standardize Editors + Remaining bugs #648

sashadev-sky opened this issue Oct 27, 2020 · 7 comments

Comments

@sashadev-sky
Copy link
Member

sashadev-sky commented Oct 27, 2020

Description

I am helping out with the winter prioritization session so have been reviewing recent work. I made a parsed down summary of what's left over from the Editor Overhaul project for Research Note Editor & Wiki Editor, which could really be standardized. I ended up with a lot of bullets, so decided to remove the specifics from the session discussion because it's overkill and move it here.

This wasn't signed off by anyone so feel free to change the "[Planning]" title - it's just building on the work Emily did on standardizing the toolbars.

Linking this to #630 because it is ,most likely in some way interdependent with it.

Task List

These should all be fixed, but BOLD bullets are in my opinion showstoppers.

PublicLab.Editor:

Research Note Editor:

Wiki Editor:

  • Save and Recover buttons don’t work properly
  • Can’t add tags before publishing
  • An error creating a new wiki gets you redirected to a form with the Research Editor interface (and publishing breaks)
    If you publish a wiki with a duplicate title, you will lose all you work and be redirected to the other wikis page
  • Revert button doesn't work properly
  • Can’t delete a wiki that has more than 1 contributor
  • If you publish a wiki with a title that already exists, on publish you will be redirected to that existing wiki and lose all you work
@welcome
Copy link

welcome bot commented Oct 27, 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.

@sashadev-sky
Copy link
Member Author

@cesswairimu @ebarry @jywarren @publiclab/comment-maintainers @publiclab/editor

@sashadev-sky
Copy link
Member Author

Looking for some input here about availability and thoughts. I think it would be "small wrap-up contract for editor overhaul"?

@ebarry
Copy link
Member

ebarry commented Oct 27, 2020

Hi @sashadev-sky it's great to have a fresh look across all this.
For all, i wish to note that the wiki deletion behavior may not have been previously documented.

@sashadev-sky can you coordinate with @sagarpreet-chadha on the short list that he assembled and is working from this month? #627 and https://pad.publiclab.org/p/editor-tasks? Are any of the ones Sasha highlighted NOT in there already? Thank you all!🌳

@sagarpreet-chadha
Copy link
Contributor

Hey @sashadev-sky ,
Adding to @ebarry comment above, I have made tracking issue here:
#627 (comment)
Thanks 😄

@jywarren
Copy link
Member

jywarren commented Oct 27, 2020

Hi all, this is great, thank you!

Numbering is reset ...

Noting that we'd suggested a GitHub Markdown style fix but @sashadev-sky also discovered a workaround

Custom insert doesn’t properly insert a wiki as a grid

Let's open a breakout issue for this and see if @keshav234156 has thoughts on it? Perhaps we can start with a test demonstrating the issue. Thanks!

Test example: https://github.com/publiclab/PublicLab.Editor/blob/main/test/ui-testing/CustomInsert.test.js#L8

Also what was the specific issue with the toolbar? Noting theres work on this in #600 #599 #496 #522 and others, so may need to clarify what the problem + solution are, since the original issue flipped back and forth a few times due to varying user preferences. Thanks!

@sashadev-sky
Copy link
Member Author

@ebarry none of these are on #627

@jywarren jywarren added this to the v3.1 milestone Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants