Re: [pgAdmin4][Patch]: Added the shortcut (CTRL + 0) to reset the zoom level in runtime

On Tue, Jun 13, 2017 at 2:52 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Nice work! Committed (along with an update to the docs).
>
> Congratulations on your first C++ patch :-)
>
> Thanks Dave. :)

> On Tue, Jun 13, 2017 at 7:07 AM, Khushboo Vashi
> <khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> > Hi,
> >
> > On Mon, Jun 12, 2017 at 5:27 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> >>
> >> Hi
> >>
> >> On Mon, Jun 12, 2017 at 5:33 AM, Khushboo Vashi
> >> <khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> >> > Hi,
> >> >
> >> > Please find the attached patch to reset the zoom level in the desktop
> >> > runtime.
> >> >
> >> > Added the new shortcut CTRL + 0 to reset the zoom level.
> >>
> >> The code looks good, however I don't think resetting to the startup
> >> value is really what we want. Shouldn't it reset it to 1.0?
> >>
> > I have changed the code.
> > Also, introduced a single slot for Zoom In/Out/Reset rather than 3
> > different.
> >>
> >> --
> >> Dave Page
> >> Blog: http://pgsnake.blogspot.com
> >> Twitter: @pgsnake
> >>
> >> EnterpriseDB UK: http://www.enterprisedb.com
> >> The Enterprise PostgreSQL Company
> >
> >
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>