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

Message formatting (Bold, Italic, URLs, @person) #224

Merged
merged 18 commits into from
Oct 30, 2022

Conversation

ouchadam
Copy link
Owner

@ouchadam ouchadam commented Oct 24, 2022

Add html message parsing with some simple rendering

  • Headers are bold text with line break

2022-10-30T15:11:04,509505309+00:00

  • Lists of items are lines prefixed with - or a number if the list is ordered

2022-10-30T15:11:10,783540942+00:00

  • Supports bold and italic (not together though)

2022-10-30T15:10:56,381981577+00:00

  • Mention a user

2022-10-30T15:10:44,229789393+00:00

  • Clickable direct urls or named hyperlink [a link](https://url.com)

2022-10-30T15:10:50,607202795+00:00

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2022

APK Size: 1.99 MB

@codecov
Copy link

codecov bot commented Oct 28, 2022

Codecov Report

Merging #224 (7e7f17a) into main (c301563) will increase coverage by 0.98%.
The diff coverage is 77.45%.

@@             Coverage Diff              @@
##               main     #224      +/-   ##
============================================
+ Coverage     57.74%   58.73%   +0.98%     
- Complexity      519      587      +68     
============================================
  Files           224      231       +7     
  Lines          5633     5893     +260     
  Branches        650      697      +47     
============================================
+ Hits           3253     3461     +208     
- Misses         2089     2125      +36     
- Partials        291      307      +16     
Impacted Files Coverage Δ
...ngine/src/main/kotlin/app/dapk/st/engine/Models.kt 56.52% <ø> (ø)
core/src/main/kotlin/app/dapk/st/core/RichText.kt 0.00% <0.00%> (ø)
.../kotlin/app/dapk/st/domain/sync/RoomPersistence.kt 53.33% <ø> (ø)
...in/kotlin/app/dapk/st/messenger/MessengerScreen.kt 0.00% <ø> (ø)
...kotlin/app/dapk/st/messenger/MessengerViewModel.kt 32.98% <0.00%> (ø)
...k/st/notifications/RoomEventsToNotifiableMapper.kt 100.00% <ø> (ø)
...main/kotlin/app/dapk/st/engine/DirectoryUseCase.kt 0.00% <0.00%> (ø)
...in/kotlin/app/dapk/st/engine/SendMessageUseCase.kt 0.00% <0.00%> (ø)
...k/st/matrix/message/internal/SendMessageUseCase.kt 96.90% <ø> (ø)
...c/main/kotlin/app/dapk/st/matrix/sync/RoomState.kt 65.21% <ø> (ø)
... and 13 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ouchadam ouchadam force-pushed the feature/message-formatting branch from 24706f2 to 55745b9 Compare October 29, 2022 09:12
@ouchadam ouchadam merged commit 3df8c0b into main Oct 30, 2022
@ouchadam ouchadam deleted the feature/message-formatting branch October 30, 2022 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant