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

[Story] RTE Rich text mode on EX #1937

Closed
10 tasks done
Tracked by #1940
langleyd opened this issue Jul 27, 2023 · 9 comments
Closed
10 tasks done
Tracked by #1940

[Story] RTE Rich text mode on EX #1937

langleyd opened this issue Jul 27, 2023 · 9 comments
Assignees
Labels
A-Rich-Text-Editor Issues with the new rich text editor, also known as the WYSIWYG editor App: ElementX Android App: ElementX iOS T-User Story Team: Element X Feature X-Needs-Signoff Stories and Epics which are ready for review by product, design and QA

Comments

@langleyd
Copy link
Member

langleyd commented Jul 27, 2023

Description

  • As a user
  • I want to format message content
  • so that I can add structure to my longer messages.

Size estimate

M

Dependencies

Acceptance criteria

Formatting buttons

Message content can be formatted with:

  • Inline Formatting (Bold, Italic, underline, strikethrough, code)
  • Lists(unordered and numbered)
  • Blocks (Quotes, Codeblocks)

Hide and show formatting buttons

  • Enable rich text formatting buttons from the action bottom sheet
  • Quickly hide rich text formatting options with the X button

Message output

When the user presses the send button, the app should send a message event with both: formatted_body containing the HTML output of the rich text editor and body containing the markdown formatted output of the rich text editor.

Out of scope

Sign-off

Android

  • Design sign-off on completion
  • QA sign-off on completion
  • Product sign-off on completion

iOS

  • Design sign-off on completion
  • QA sign-off on completion
  • Product sign-off on completion

Subtasks

Android

  1. A-Composer A-Timeline O-Occasional S-Major T-Defect X-Needs-Rust
    jonnyandrew
  2. Android
    jonnyandrew

iOS

  1. Trigger-PR-Build
    alfogrillo

Other

  1. jonnyandrew

Links

@jonnyandrew
Copy link

@alfogrillo @langleyd I've pulled the error tracking requirement for iOS out of this story into a separate issue to be prioritised separately:

@langleyd langleyd added the X-Needs-Signoff Stories and Epics which are ready for review by product, design and QA label Sep 19, 2023
@jonnyandrew jonnyandrew added the A-Rich-Text-Editor Issues with the new rich text editor, also known as the WYSIWYG editor label Oct 5, 2023
@callumu
Copy link

callumu commented Oct 12, 2023

iOS: code block and inline code colours need to be updated to the designs

Android: there is still the border radius issue when moving from one line onto two lines. Code block spacing around the mono font is inaccurate in the composer. Code block and inline code colours are inconsistent in the composer vs on the timeline

@alfogrillo
Copy link

iOS: code block and inline code colours need to be updated to the designs

@callumu can you be more specific on this?
What's wrong? The background color looks in line with the design ("color gray 100")

@callumu
Copy link

callumu commented Oct 12, 2023

iOS: code block and inline code colours need to be updated to the designs

@callumu can you be more specific on this? What's wrong? The background color looks in line with the design ("color gray 100")

Sorry yes.

On dark mode the text colour for code blocks is the same colour as the background making the text impossible to see.

I also wanted to note that we discussed the difficulty getting inline code and code blocks to match the designs , and it's something to work on when there is more time.

@callumu
Copy link

callumu commented Oct 12, 2023

I also noticed on Android that the mono font text size is different for code blocks vs inline code

@callumu
Copy link

callumu commented Oct 12, 2023

I think the code block colour error is a bug:

RPReplay_Final1697126127.mp4

@alfogrillo
Copy link

On dark mode the text colour for code blocks is the same colour as the background making the text impossible to see.

Just for the record: this seems not to be a color issue, but a bug on the RTE.
Notice how the color changes while typing (especially on new lines).

Simulator.Screen.Recording.-.iPhone.14.-.2023-10-12.at.17.55.20.mp4

@alfogrillo
Copy link

I think the code block colour error is a bug:

You found the same problem! 😄

@kittykat
Copy link
Contributor

Any outstanding tasks will be added to element-hq/element-x-ios#1702

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rich-Text-Editor Issues with the new rich text editor, also known as the WYSIWYG editor App: ElementX Android App: ElementX iOS T-User Story Team: Element X Feature X-Needs-Signoff Stories and Epics which are ready for review by product, design and QA
Projects
None yet
Development

No branches or pull requests

5 participants