After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 768004 - Allow changing font colours and sizes
Allow changing font colours and sizes
Status: RESOLVED OBSOLETE
Product: gnome-documents
Classification: Core
Component: books
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Books Maintainers
GNOME documents maintainer(s)
Depends on: 768003
Blocks:
 
 
Reported: 2016-06-24 11:34 UTC by Bastien Nocera
Modified: 2021-07-05 11:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
epubview: font size and invert colors buttons (5.67 KB, patch)
2016-06-30 08:28 UTC, Daniel Garcia
reviewed Details | Review

Comment 1 Daniel Garcia 2016-06-30 08:28:05 UTC
Created attachment 330641 [details] [review]
epubview: font size and invert colors buttons

I've added a new menu to control font options. In this menu I've added the
font size buttons and another button to invert colors.

The invert colors assume that the background is light and the text is dark
and when it's inverted sets the background to black and the text to white.

The menu is ugly now, if we can create some buttons or widgets in this
menu, it will look a lot better.
Comment 2 Daniel Garcia 2016-06-30 08:43:52 UTC
This patch is a first version. I've some problems creating the menu, I wanted to add buttons and not close the menu on click but I don't know how. In any case I've uploaded this patch.
Comment 3 Cosimo Cecchi 2016-06-30 18:49:00 UTC
Review of attachment 330641 [details] [review]:

Thank you, Daniel.
I pushed your patch in a branch here, together with some further cleanups that streamline the UI to make it similar to the existing night mode of the Evince view: https://git.gnome.org/browse/gnome-documents/log/?h=wip/epub-colors

The main issue remaining is that the invert colors function should probably not hardcode black and white like that, and does not really seem to work reliably. I tried playing a little bit with the filter: invert CSS function and it seems to help somewhat, but I couldn't find a convincing combination yet. Help welcome, feel free to push to the branch :)
Comment 4 Bastien Nocera 2016-06-30 18:58:37 UTC
Wouldn't it be better if this functionality was hidden inside the view provided by libgepub?
Comment 5 Cosimo Cecchi 2016-06-30 19:06:56 UTC
Definitely, but until we have a good implementation it doesn't really matter, and I'm fine keeping it in the Documents JS code as it makes it easier to hack.
Comment 6 Debarshi Ray 2018-02-01 12:55:29 UTC
Ping. Anybody interested in moving this forward?
Comment 7 GNOME Infrastructure Team 2021-07-05 11:31:53 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new ticket at
  https://gitlab.gnome.org/GNOME/gnome-documents/-/issues/

Thank you for your understanding and your help.