PrimeDAO / prime-deals-dapp

DAO to DAO negotiations facilitated, ratified, and archived.
https://www.prime.xyz/deals
3 stars 1 forks source link

Feature/1061 clause enhancements #1134

Closed testing-life closed 1 year ago

testing-life commented 1 year ago

What was done

image image

Testing

Before

After

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
prime-deals-dapp ✅ Ready (Inspect) Visit Preview Jul 27, 2022 at 8:32AM (UTC)
hiaux0 commented 1 year ago

Overall, the UI integration looks clean! Thanks for picking the task up :) Was prob not straightforward to figure out the integration.

Happy coding (supportive message goes here) :sun_with_face:


Fnality


Test text:

bold, italic, underline

copy paste

image image

hiaux0 commented 1 year ago

Could you add into the initial comment

  1. why ckeditor was chosen?
  2. what format the formatted text is saved with?
blink-km commented 1 year ago

copy paste from other sources should be supported

testing-life commented 1 year ago

changes addressed:

testing-life commented 1 year ago

why ckeditor was chosen? documentation, freshness of packages, ease of styling, available plugins. oh and i got it working the easiest :P

what format the formatted text is saved with? it's a string containing the text and formatted words wrapped in HTML elems (i,u,b, ul>li, etc)

testing-life commented 1 year ago
  1. Probably best if you can use the palette colors from the design system for all elements rather than fixed HSL, hex etc. Please note that there are also variables for sizes, such as border-radius and padding that you can use.

    1. I get an error after trying to open a link I've entered. Maybe you can find a fix for that?

    2. Please make the hover-border change through the whole element and not only the lower half. If it requires a longer time to refactor, maybe it's better to triage it because it would probably come back to us during testing.

    3. Please revert the quotes in all files to double quotes... It appears to apply all .scss files.

    4. Please don't hate me for being picky! joy

ad1. fixed

ad2. resolved image

ad3. fixed image

ad4.fixed

ad5. that's ok, i am capable of hating people for several things simultaneously :P

testing-life commented 1 year ago

ad1. reinstated custom css. overriding some custom props from the component turned out impossible, plus i do think we should look into custom props grolbally, as per the telegram chat ad2. removed ad3. querySelector now uses index, so displays multiple editors.