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

[NARUDESIGNS]

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

• Resolve questions on transforming Jasmine to Jest tests
• Port more Jasmine files to Jest test
• Remove jasmine files that have been converted to jest test
• fixed row and column subtract button values for issue #755 #757
• Added style to col subtract button to fix issue #756 #758
• Converted errors jasmine test to jest test fixes #770 #771
• Converted formatter jasmine file to jest test file fixes #774 #775
• converted editor_spec test to jest test fixes #772 #773
• Converted wysiwyg jasmine file to jest test fixes #784 #785
• Converted Tags module jasmine file to jest test fixes #787 #788
• Converted prepopulated jasmine file to jest test fixes #790 #789
• Converted map module jasmine file to jest test fixes #719 #792
• Rich text module jest test conversion #803
• Custom Insert popover hide on clicking on anything on the screen fixe… #809 (in collaboration with @RaviAnand111)
• Table popup hide on click on page and everything except table button … #800 (in collaboration with @RaviAnand111)
• converted history jasmine test to jest #815
• Converted mainImageModule to jest test #818
• refined custom insert dropdown UI fixes #816 #817
• Address spacing and padding issues across the editor page for congested elements. For example, buttons and values in the table popover are tightly packed together
• Style buttons and input fields in popovers to sync with the other buttons and inputs on the page
• Clean up jasmine files #826
• updated version number in package.json and package-lock.json files #833
• Hiding Table popover on pressing esc button #820 (in collaboration with @RaviAnand111)
• Hiding Custom Insert Popover on Pressing esc button #821 (in collaboration with @RaviAnand111)
• Permanently show tip, modify tip to clearly express that the use of tab key 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 space e.g "next UI"
• refined mainImage section to account for responsiveness fixes #842 #843
• Refine the editor section
• Style editor tool buttons to match each other for consistency, centre it on mobile

FTO

• Add tooltip to Custom Insert button #782
• Fixed style on dropdown button in custom insert popover #802
• added changes for consistent design #836

[NARUDESIGNS]

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]

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]

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]

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]

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>.

[NARUDESIGNS]

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]

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]

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]

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: