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

Code visualization with inline code and code blocks buggy #1002

Closed
HagenMorano opened this issue Dec 22, 2020 · 9 comments
Closed

Code visualization with inline code and code blocks buggy #1002

HagenMorano opened this issue Dec 22, 2020 · 9 comments

Comments

@HagenMorano
Copy link

Describe the bug
There are basically two issues with the coloring and displaying of inline code and code blocks (markdown):

  1. Inline Code (`example`) gets cut in view mode when too long (see screenshot 2)
  2. Color contrast of code blocks is very low on both view and edit mode (only tested on dark mode, see screenshot 1 & 2), also there are some dark ugly fragments in view mode (screenshot 2)
    ```
    example
    ```

To Reproduce
Steps to reproduce the behavior:

  1. Create a long inline code
  2. Create a code block in dark mode

Expected behavior
Concerning inline code: Line should break
Concerning code block: Colors should be more contrasting

Screenshots

Edit mode:

Screenshot_2020-12-22-11-25-50-709_it niedermann owncloud notes

View mode:

Screenshot_2020-12-22-11-26-00-034_it niedermann owncloud notes

Smartphone

  • Nextcloud Notes-Version (android app): 2.17.1
  • F-Droid or Play Store: Play Store
  • Android-Version: 10
  • Device: Xiaomi Mi Mix 2s

Server

  • Nextcloud version: 20.0.3
  • Nextcloud Notes version (server app): 4.0.1
@stefan-niedermann
Copy link
Member

stefan-niedermann commented Dec 24, 2020

Uhmm 🤔 i can't reproduce this issue. On my device as well as on any emulator i use, the colors are correct.

Can you maybe tell me more about your Android settings like system wide dark mode, automatic night mode, display color adjustments and so on? Have you changed any default configuration of your Android system (to be clear: I am not talking about the App settings)?

Is the text readable in light mode? Which theme have you set in the app? "Dark" or "System default"?

@stefan-niedermann
Copy link
Member

cc @dan0xii

@HagenMorano
Copy link
Author

HagenMorano commented Dec 25, 2020

The only relevant setting I can see is the global dark mode. If set to dark, switching the theme in the application has no effect (dark,light and system default all result in a dark mode), switching global dark mode to light results in a desired behaviour: light and system default result in a light theme, dark results in a dark theme (expected behaviour).

Light theme looks good regarding colors but long inline codes are also cut.

There is one more UI relevant setting I can see in the dev settings: Turn on MIUI optimization which also seems to have no effect.

Also I don't remember changing any system settings concerning colors etc. (except dark mode)

Maybe the problem gets resolved within #549 ?

@stefan-niedermann
Copy link
Member

Maybe the problem gets resolved within #549 ?

I hope so, though the preview mode will still take some time to get implemented. But it would be very helpful if you could check the behavior of 2.18.1 (available at the Play Store Beta channel) to see whether this already fixes the issue at the editor. 🙂

So long, stay well and merry X-Mas 🙂

@HagenMorano
Copy link
Author

Thanks for the quick reply! Just checked out beta version, but no changes concerning this issue unfortunately.

The issue is no show stopper as I am personally using backtick comments on laptop which is working quite good.

Merry Christmas to you too 🎄

@stefan-niedermann
Copy link
Member

Would you mind adding a screenshot of v2.18.1 editor?

@HagenMorano
Copy link
Author

HagenMorano commented Dec 25, 2020

Sure - after restarting application I just realized that edit mode now is indeed looking better - also font is now monospace. Like! See screenshots

Screenshot_2020-12-25-18-34-27-432_it niedermann owncloud notes
Screenshot_2020-12-25-18-34-17-014_it niedermann owncloud notes

@stefan-niedermann
Copy link
Member

Should be fixed in v2.19.0 (available soon at the Play Store beta channel and on F-Droid). Please reopen this issue in case it is still a problem with this version 🙂

And thanks for the helpful screenshots!

@HagenMorano
Copy link
Author

Confirm 👍 Thanks a lot!

1024-2048
1024-2048 (1)

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

No branches or pull requests

2 participants