publiclab / PublicLab.Editor

A general purpose, JS/Bootstrap UI framework for rich text posting. An author-friendly, minimal, mobile/desktop interface for creating blog-like content, designed for PublicLab.org
https://publiclab.github.io/PublicLab.Editor/examples/
GNU General Public License v3.0
80 stars 128 forks source link

[Planning] Refine the Public Lab Rich Editor (Outreachy) #760

Open NARUDESIGNS opened 2 years ago

NARUDESIGNS commented 2 years ago

This issue contains a list of tasks for my project. This is list is subject to change as some tasks might be added later on or removed.

My Outreachy Proposal

PROJECT TASKS

FTO

NARUDESIGNS commented 2 years ago

Hello, @TildaDares @jywarren @Tlazypanda kindly review my tasks list. Also just to clarify please, I don't have to tackle them based on the order they are listed, right? Thank you!

TildaDares commented 2 years ago

Great work on breaking your tasks into smaller sub-tasks @NARUDESIGNS!

  • Rename tags section, centre elements, style input field to match other input fields. Permanently show tip, modify tip to clearly express that the use of tab or comma affirms a tag. Also, state that multiple-word tags should be separated using the hyphen (dash) symbol e.g "next-UI" instead of "next UI"

There's a lot going on with this task. Do you think maybe this could be broken up further?

TildaDares commented 2 years ago

I just found out that you can convert a task list to an issue by using the Convert to issue button. That's cool!

NARUDESIGNS commented 2 years ago

Great work on breaking your tasks into smaller sub-tasks @NARUDESIGNS!

  • Rename tags section, centre elements, style input field to match other input fields. Permanently show tip, modify tip to clearly express that the use of tab or comma affirms a tag. Also, state that multiple-word tags should be separated using the hyphen (dash) symbol e.g "next-UI" instead of "next UI"

There's a lot going on with this task. Do you think maybe this could be broken up further?

Yes, I agree with you @TildaDares. After the meeting yesterday, I realised that some of my task lists need to be broken up to further tasks. I remember @jywarren mentioned that ideally, a PR should do one thing but I was also worried that it will be a very long list of PRs if each PR did just one thing.

I've however broken down that task and will go through again to see which task needs to be broken down.

jywarren commented 2 years ago

It's ok to end up with a long list of PRs, although perhaps it's good to start with only 3-5 max open at a time?

On Tue, Dec 7, 2021, 6:15 AM Paul Ibeabuchi C. @.***> wrote:

Great work on breaking your tasks into smaller sub-tasks @NARUDESIGNS https://github.com/NARUDESIGNS!

  • Rename tags section, centre elements, style input field to match other input fields. Permanently show tip, modify tip to clearly express that the use of tab or comma affirms a tag. Also, state that multiple-word tags should be separated using the hyphen (dash) symbol e.g "next-UI" instead of "next UI"

There's a lot going on with this task. Do you think maybe this could be broken up further?

Yes, I agree with you @TildaDares https://github.com/TildaDares. After the meeting yesterday, I realised that some of my task lists need to be broken up to further tasks. I remember @jywarren https://github.com/jywarren mentioned that ideally, a PR should do one thing but I was also worried that it will be a very long list of PRs if each PR did just one thing.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/publiclab/PublicLab.Editor/issues/760#issuecomment-987827398, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAF6J2QLD7JHR67MHFIJSTUPXUGFANCNFSM5JOWPNZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

NARUDESIGNS commented 2 years ago

It's ok to end up with a long list of PRs, although perhaps it's good to start with only 3-5 max open at a time? On Tue, Dec 7, 2021, 6:15 AM Paul Ibeabuchi C. @.***> wrote: Great work on breaking your tasks into smaller sub-tasks @NARUDESIGNS https://github.com/NARUDESIGNS! - Rename tags section, centre elements, style input field to match other input fields. Permanently show tip, modify tip to clearly express that the use of tab or comma affirms a tag. Also, state that multiple-word tags should be separated using the hyphen (dash) symbol e.g "next-UI" instead of "next UI" There's a lot going on with this task. Do you think maybe this could be broken up further? Yes, I agree with you @TildaDares https://github.com/TildaDares. After the meeting yesterday, I realised that some of my task lists need to be broken up to further tasks. I remember @jywarren https://github.com/jywarren mentioned that ideally, a PR should do one thing but I was also worried that it will be a very long list of PRs if each PR did just one thing. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#760 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAF6J2QLD7JHR67MHFIJSTUPXUGFANCNFSM5JOWPNZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Alright, thank you.

jywarren commented 2 years ago

I just found out that you can convert a task list to an issue by using the Convert to issue button. That's cool!

OMG I did NOT know that was a button for the longest time!!!! Amazing! But... maybe a UI fail? lol! ❤️

NARUDESIGNS commented 2 years ago

I just found out that you can convert a task list to an issue by using the Convert to issue button. That's cool!

OMG I did NOT know that was a button for the longest time!!!! Amazing! But... maybe a UI fail? lol! ❤️

😂 It has a tool tip but we also missed that.

jywarren commented 2 years ago

This checklist is looking great @NARUDESIGNS !!

I was admiring this nice minimal way to hide the text editing options in the GitHub comment editor, btw, thought I'd add it here as inspiration:

image