Re: [PATCH] Toolbar button to control zoom level

Rodrigo Arias <[email protected]>
Newsgroups gmane.comp.web.dillo.devel
Message-ID <[email protected]>
Hi,

> Here is an updated patch with a better icon and fixed tooltip.
> Hopefully somebody finds it useful :)

Thanks! I applied it from the attached file, like this:

$ cd src
$ md5sum /tmp/zoom-button.patch
c8d48b41f30ba889ddd610860ecdc310  /tmp/patches/zoom-button.patch
$ patch -p1 < /tmp/zoom-button.patch
patching file pixmaps.h
patching file ui.cc
patching file ui.hh

In the future, please perform the diffs from the top level directory, 
like so:

$ diff -up dillo-orig/ dillo-mod/

(Or better, via "git format-patch" if you use git)

>all that is needed now is the 'magnifying glass' icon above the word 
>Zoom 'borrowed' from the 'search button' :)

There is a button there, but with the default Dillo options the color of 
the icon is very similar to the background (attached screenshot) and is 
barely visible. I think Alex may be using a different color scheme.

I'm thinking if we could hide the save button by default and put the 
zoom there instead, as save is available from the page menu either way, 
and I think the zoom button would be used more frequently.

We probably should add a "show_zoom" preference too in dillorc. Also we 
may want to use the middle click to reset the zoom level.

Best,
Rodrigo.

_______________________________________________
Dillo-dev mailing list -- dillo-dev-lx9mn2B4QYRWk0Htik3J/[email protected]
To unsubscribe send an email to dillo-dev-leave-lx9mn2B4QYRWk0Htik3J/[email protected]
zoom.png (image/png, 2.7 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.