Kiwix-android: Negative Image on Help Screen in night mode

Created on 16 Feb 2018  路  13Comments  路  Source: kiwix/kiwix-android

Is it a ?

  • [x] Bug Report

  • [ ] Feature Request

Bug Report

Step 1: Are you ready to report ?

  • Your issue may already be reported! Please search on the issue track before creating one.
  • [x] I have test with the last (dev) version of the software? ( If not, and if possible, run your test again with the most recent version available.)

Step 2 : What is your environment ?

  • browser / app : App
  • version of the software you use : Version 2.3 Build: 55
  • device / operating system : Android 7.0 (Samsung Galaxy S6)

Step 3: Describe the problem:

  • What do you obtain? A copy of the error message or a screenshot is always useful
    In the night mode, the image on the help screen turns negative:
    screenshot:
    screenshot_20180216-102025
    This problem is only in this image. There is no problem in the images from wikipedia
  • When does this occur?
    In the night mode only

    Steps to reproduce:

  • enable night mode in settings

  • Go to the help screen

  • What should be the behaviour from your point of view? How do you expect the service to work?
    I think the color of the image should not be negative.

Step 4 (optional) : Other Comments:

  • What is the context of this ticket? If not obvious, explain why you need to do this.

  • If you have an idea about the technical background of the ticket, please share it.

Feature Request

Feature Description

Please describe the feature you want to add to the project.

enhancement

All 13 comments

I would like to work on this issue.

I think it's better imo as there is less blinding white. Anyone else have any opinions?

Now that you pointed it out, I guess you are right; the white background would sort of ruin the whole purpose of the night mode...

If you don't like the look you could try recolouring it to look better while keeping minimal white else we can close this.

sure I will tinker around with it on photoshop, and see if I can improve it in some way...

I think that this is probably the best solution. I thought of making the background grey instead of brown, but I think that brown gave a better overall highlight to the image... So I guess we can close this issue then.

@mhutti1 @kelson42 @srv-twry What do you think about this?

new

Adding a blue light filter not only makes it easier for the eyes but also helps in retaining the original feel of the image that was somehow lost in making it negative...

Do you suggest some other edits?

Here are the images:
original:
kiwix_welcome
new (for night mode):
sid

I can remove the dark high light from the ends if you like...

Looks good to me.

@mhutti1 @srv-twry @kelson42 , to do this task, I was thinking of using Data Binding (https://developer.android.com/topic/libraries/data-binding/index.html) in the help.xml layout file to set the src of the image based on the boolean value of the nightMode variable from userPreferences.

Do you think this is the correct way, or is there any other better way.

@siddharth2010 I have never used data bindings before but they look interesting. If you want to try to use them then sure go ahead and I am sure I will learn something.

@mhutti1 @siddharth2010 I have used Data bindings before and it's a really good library to use. Although IMO we should either use Data bindings in each and every layout or shouldn't use it all. There should be consistency throughout the application.

Edit: Moreover the error messages from the library is extremely tough to decrypt.

Was this page helpful?
0 / 5 - 0 ratings