Hi Please find updated patch.
Thanks, Surinder On Thu, Jun 8, 2017 at 7:16 PM, Dave Page <dp...@pgadmin.org> wrote: > Hi > > On Thu, Jun 8, 2017 at 5:38 AM, Surinder Kumar > <surinder.ku...@enterprisedb.com> wrote: > > Hi > > > > The attribute key for Ok, Save buttons should be mapped to 13(Enter), > > for Help - 112(F1) and > > for Cancel - 27(Esc) > > The above changes are made into all dialog related template files. > > > > PFA patch and review. > > Looks good. Just a couple of thoughts: > > - You've missed the object property dialogues. This really needs to be > consistent across all dialogues :-) > > - The behaviour with Enter is a bit odd, because priority will be > given the the focussed control to handle it. That's what we want if > it's a text area for example, but on the other hand, if you open the > Preference dialogue, then the Help button gets focus first, so Enter > there will end up opening the docs. I suspect in most cases the > behaviour you've got is fine, so I'm not inclined to change it. <plays > a little> Hmm, I see similar behaviour in some other apps. Let's leave > it as you've done it :-) > > Please add support for the properties dialogues, then I think we're good > to go. > Done. > > Thanks! > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company >
Feature_1535_v1.patch
Description: Binary data
-- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers