Re: Unreviewed Patches

[Marco Pesenti Gritti <mpgritti gmail com>]

> > 170111        Toolbar in fullscreen mode doesn't respect Btoolbar
> > settings
> >       http://bugzilla.gnome.org/show_bug.cgi?id=170111
> 
> I don't understand why we're overriding this to begin with.  Marco, do
> you know the history of this line?

Fullscreen was ported from epiphany, I guess this was so that the
toolbar would take less vertical space. Though it doesnt make much
sense when the user choosen mode is icon/text only.

CCing Christian

> Before we release, we need to fix the find code to be less buggy.  This
> might take another few (small) changes to ev_document.h, but Kristian
> and I know how to fix it.  Is there anything else that needs landing
> here?

I think clipboard is a very important part of evince functionalities
and we should try to get it back even if real text selection is not
possible at the moment. I will see  if that's doable with the new
architecture.

Marco