From: | Andreas Pflug <pgadmin(at)pse-consulting(dot)de> |
---|---|
To: | ian(at)uns(dot)ns(dot)ac(dot)yu |
Cc: | pgadmin-support(at)postgresql(dot)org, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgadmin-support] Patch: view data for tables/views on double |
Date: | 2004-09-16 10:06:52 |
Message-ID: | 414965BC.4000403@pse-consulting.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers pgadmin-support |
Ivan Nejgebauer wrote:
First of all, this is certainly hackers stuff, please post on
pgadmin-hackers!
> The attached patch a) defines a double click handler for the list in the
> properties pane, so that a double click on an item
You're right, doubleclick is missing on the property pane.
in the list launches
> the properties editor for the item (or the data view if the item is a
> table or a view); and b) modifies the tree view double click handler so
> that a double click on a table or a view launches the data view for the
> corresponding table/view.
This isn't acceptable like this, because it introduces different
behaviour between clicking on a "properties" item and a tree item. Both
must behave identically. Actually, not only double click is missing, but
also right mouse click.
>
> Rationale for different behavior with tables/views:
This is reasonable, but before that some prerequisites have to be met:
- the default action should be visible to the user, preferrably
displayed bold in the context menu.
- the default action might differ from user to user. For sure, an
unconditional "View Data" on a table is a bad idea. I already hear those
guys yelling double clicking on a 100 million row table...
Regards,
Andreas
From | Date | Subject | |
---|---|---|---|
Next Message | cvs | 2004-09-16 10:07:42 | CVS Commit by dpage: Updated todo. |
Previous Message | cvs | 2004-09-16 10:04:07 | CVS Commit by dpage: The password text field in the "Add Server" dialog |
From | Date | Subject | |
---|---|---|---|
Next Message | Andreas Pflug | 2004-09-16 10:07:17 | Re: Patch: password style for the "Add Server" |
Previous Message | Dave Page | 2004-09-16 10:05:53 | Re: Patch: password style for the "Add Server" dialog |