Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: add a wrap option into the settings #26

Closed
petersirka opened this issue Jul 8, 2015 · 15 comments
Closed

UI: add a wrap option into the settings #26

petersirka opened this issue Jul 8, 2015 · 15 comments
Labels

Comments

@petersirka
Copy link

I work with Postico everyday and it would be great if the Postico settings will contain the wrap option for headers and values with a minimal column width e.g. 120px. It's better for reading.

screenshot

What do you think about it?
Thanks.

@jakob
Copy link
Owner

jakob commented Jul 8, 2015

I'm not sure I understand correctly. Do you suggest that Postico should clip the column title (like this: idcatgo...) when the column is less than 120px wide?

I agree that wrapping column titles are not pretty, but I always found clipped column titles extremely annoying.

@petersirka
Copy link
Author

idcatgo... yes and you're right. I don't want to add it as default settings, only as option. Developers who work with DB know their columns. Thanks.

PGAdmin:

screen shot 2015-07-08 at 9 07 07

SQLPro for MSSQL
screen shot 2015-07-08 at 9 06 25

@danillos
Copy link

danillos commented Jul 8, 2015

Hi @jakob, I prefer idcatgo... too.

@jopotts
Copy link

jopotts commented Jul 14, 2015

I would prefer no wrapping too. More importantly, the fields in the table should not wrap in my opinion, and rows should be a single char high, not two. You have the right sidebar to view details if needs be so no need to clutter the table. Also, the grey/white stripe of Sequel Pro's table is nice rather than dividing with a line. Thanks for the great software!

@jakob
Copy link
Owner

jakob commented Jul 21, 2015

I've redesigned the table view a bit, and turned off wrapping in the column headers.
http://eggerapps-downloads.s3.amazonaws.com/postico-2015-07-21.zip

I'm considering adding a preference for the row height, so that people can also turn off wrapping in the content area. That's a bit more work, so I'm saving it for later.

@jopotts
Copy link

jopotts commented Jul 21, 2015

The headers are looking good. Please please make the table layout single line (with grey banding). Make it like Finder list view. No need for a preference setting even. Postico is the best postgres app available for simply browsing data, but this would really be the icing on the cake! Thanks!

@himikk
Copy link

himikk commented Sep 30, 2015

Please make grid rows single line. It's very critical.

@petersirka
Copy link
Author

@himikk +1

@rushire
Copy link

rushire commented Oct 16, 2015

Agree with @himikk, please fix, this is very annoying.

@wellsoliver
Copy link

Agreed but in a polite and loving manner

@jakob
Copy link
Owner

jakob commented Feb 4, 2016

I've now added an option to configure the number of lines per row in the table view, so you can change it to a single line. Prerelease build:
https://eggerapps-downloads.s3-eu-west-1.amazonaws.com/postico-1309.zip

I'm planning to add an option for alternating row colors instead of grid lines as well, but I first need to redesign the preference window which ended up pretty messy.

@jakob jakob added the started label Feb 4, 2016
@petersirka
Copy link
Author

@jakob GREAT, really great!
Thanks

@jopotts
Copy link

jopotts commented Feb 5, 2016

Lovely! Thank you! Preference window seems fine as it is for now. Alternating colors will be nice...

Comparison with SequelPro: http://f.cl.ly/items/3k0R2N2c3f1R080p1s2j/Image%202016-02-05%20at%2012.06.53%20pm.png

@jakob
Copy link
Owner

jakob commented Feb 16, 2016

I've now implemented all the features requested here:

  1. no more wrapping in table header
  2. preference for setting number of lines per row
  3. added a pref to use alternating row background colors

I'm marking this issue as fixed now.

Please try the prerelease and let me know if this works for you!
https://eggerapps-downloads.s3-eu-west-1.amazonaws.com/postico-1334.zip

Thanks for all of your feedback!

@jakob jakob added done and removed started labels Feb 16, 2016
@jakob
Copy link
Owner

jakob commented Feb 25, 2016

This issue is resolved in Postico 1.0.4
Download here: https://eggerapps.at/postico/

@jakob jakob closed this as completed Feb 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants