Thread: 2 patches
Hi,
for personal use I've made some modifications to pgadmin. Two thnings that are very frustrating when you need to copy results to spreadsheets.
1. Decimal mark
Added an option to change default decimal mark to something other.
2. Result grid: Copy column names
Added option to include column names when copying results through clipboard
It should work with all possible selections (entire row, selective columns,...)
If you think that this changes are useful feel free to commit them.
Best regards,
Jasmin
Attachment
Hi On Sat, Apr 27, 2013 at 9:32 PM, Jasmin Dizdarevic <jasmin.dizdarevic@gmail.com> wrote: > Hi, > > for personal use I've made some modifications to pgadmin. Two thnings that > are very frustrating when you need to copy results to spreadsheets. > > 1. Decimal mark > Added an option to change default decimal mark to something other. I've applied that patch - thanks! > 2. Result grid: Copy column names > Added option to include column names when copying results through clipboard > It should work with all possible selections (entire row, selective > columns,...) That one didn't compile, however it was a relatively simple fix which I've made and committed (were you using wx 2.9 perhaps?). Thanks! -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
Hi Dave,
thank you for committing.
Attached are two "bug fixes" and an improvement.
- Can not save settings if thoussep and decimal mark are blank
- Not copying column header if selecting just one cell
Another thing is the ability to disable number formatting at all in query window. Might be useful.
2013/4/29 Dave Page <dpage@pgadmin.org>
HiI've applied that patch - thanks!
On Sat, Apr 27, 2013 at 9:32 PM, Jasmin Dizdarevic
<jasmin.dizdarevic@gmail.com> wrote:
> Hi,
>
> for personal use I've made some modifications to pgadmin. Two thnings that
> are very frustrating when you need to copy results to spreadsheets.
>
> 1. Decimal mark
> Added an option to change default decimal mark to something other.That one didn't compile, however it was a relatively simple fix which
> 2. Result grid: Copy column names
> Added option to include column names when copying results through clipboard
> It should work with all possible selections (entire row, selective
> columns,...)
I've made and committed (were you using wx 2.9 perhaps?).
Yep, I had problems with 2.8. I hope you don't have too much work with the corrections.
Thanks!
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment
Hi On Mon, Apr 29, 2013 at 4:56 PM, Jasmin Dizdarevic <jasmin.dizdarevic@gmail.com> wrote: > Hi Dave, > thank you for committing. > > Attached are two "bug fixes" and an improvement. > > - Can not save settings if thoussep and decimal mark are blank Good catch - applied. > - Not copying column header if selecting just one cell I don't think that's appropriate - the copy format should be consistent no matter how many cells are copied, and we do apply the quoting rules regardless. It might make sense to have a "Use quoting/headers with single cell" option though, to allow the user to control the behaviour (though, that name sucks - we'd need a better one). If off, we just copy the string of the cell and don't add any quotes or column names. If on, you get the column names and quotes as you would with multiple cells. > Another thing is the ability to disable number formatting at all in query > window. Might be useful. That one might be useful, but it needs some tweaking: - We don't typically have options on frmOptions for settings you can toggle directly on frmQuery. Apparently that rule wasn't followed for auto-rollback I notice though. - The menu position you've used is certainly wrong. It definitely shouldn't be in the middle of the EXPLAIN menu items, and probably shouldn't be on the Query menu at all. The View menu seems more appropriate to me. Thanks. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
2013/4/29 Dave Page <dpage@pgadmin.org>
HiGood catch - applied.
On Mon, Apr 29, 2013 at 4:56 PM, Jasmin Dizdarevic
<jasmin.dizdarevic@gmail.com> wrote:
> Hi Dave,
> thank you for committing.
>
> Attached are two "bug fixes" and an improvement.
>
> - Can not save settings if thoussep and decimal mark are blankI don't think that's appropriate - the copy format should be
> - Not copying column header if selecting just one cell
consistent no matter how many cells are copied, and we do apply the
quoting rules regardless. It might make sense to have a "Use
quoting/headers with single cell" option though, to allow the user to
control the behaviour (though, that name sucks - we'd need a better
one). If off, we just copy the string of the cell and don't add any
quotes or column names. If on, you get the column names and quotes as
you would with multiple cells.
What do you think about a dropdown box in the options dialog:
Copy column names: Never/Multiple cell selection/Always
That one might be useful, but it needs some tweaking:
> Another thing is the ability to disable number formatting at all in query
> window. Might be useful.
- We don't typically have options on frmOptions for settings you can
toggle directly on frmQuery. Apparently that rule wasn't followed for
auto-rollback I notice though.
I'm with you, it was not my prefered solution. Better would be a query window limited variable, which overrides the sys settings during one "session".
I'm going to check this again. Do we have a doxygen documentation about the source code? This would make it much easier to dive into the source code?
- The menu position you've used is certainly wrong. It definitely
shouldn't be in the middle of the EXPLAIN menu items, and probably
shouldn't be on the Query menu at all. The View menu seems more
appropriate to me.
I see. I'm going to change the position.
Thanks.
Thank you for the feedback.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Hi On Tue, May 7, 2013 at 6:53 AM, Jasmin Dizdarevic <jasmin.dizdarevic@gmail.com> wrote: > > > > 2013/4/29 Dave Page <dpage@pgadmin.org> >> >> Hi >> >> On Mon, Apr 29, 2013 at 4:56 PM, Jasmin Dizdarevic >> <jasmin.dizdarevic@gmail.com> wrote: >> > Hi Dave, >> > thank you for committing. >> > >> > Attached are two "bug fixes" and an improvement. >> > >> > - Can not save settings if thoussep and decimal mark are blank >> >> Good catch - applied. >> >> > - Not copying column header if selecting just one cell >> >> I don't think that's appropriate - the copy format should be >> consistent no matter how many cells are copied, and we do apply the >> quoting rules regardless. It might make sense to have a "Use >> quoting/headers with single cell" option though, to allow the user to >> control the behaviour (though, that name sucks - we'd need a better >> one). If off, we just copy the string of the cell and don't add any >> quotes or column names. If on, you get the column names and quotes as >> you would with multiple cells. > > > What do you think about a dropdown box in the options dialog: > Copy column names: Never/Multiple cell selection/Always That could work. >> > Another thing is the ability to disable number formatting at all in >> > query >> > window. Might be useful. >> >> That one might be useful, but it needs some tweaking: >> >> - We don't typically have options on frmOptions for settings you can >> toggle directly on frmQuery. Apparently that rule wasn't followed for >> auto-rollback I notice though. > > > I'm with you, it was not my prefered solution. Better would be a query > window limited variable, which overrides the sys settings during one > "session". Maybe, but that's not done anywhere else at the moment (having a local setting override a global one), so let's not invent a whole new way of doing things now. We should either have it on frmOptions, or frmQuery. frmQuery should be used only if there's good reason to expect this option to be toggled often. > I'm going to check this again. Do we have a doxygen documentation about the > source code? This would make it much easier to dive into the source code? No, we don't. >> >> - The menu position you've used is certainly wrong. It definitely >> shouldn't be in the middle of the EXPLAIN menu items, and probably >> shouldn't be on the Query menu at all. The View menu seems more >> appropriate to me. > > > I see. I'm going to change the position. Thanks. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company