Update to UI: Remove some borders#43
Conversation
|
I am not immediately convinced that this a good idea. I am afraid that it makes it harder to align various fields of each entry visually when looking over a large database. |
|
Agreed with Thomas. Although it looks cleaner, for those of us who have 2015-05-11 12:57 GMT-05:00 Thomas Arildsen notifications@github.com:
|
|
Is there a way to make is as an option in Preferences? Igor Fabio Steinmacher, Ph.D. On Mon, May 11, 2015 at 3:01 PM, hakova notifications@github.com wrote:
|
|
I had the same filling. Frédéric Le 11/05/2015 19:57, Thomas Arildsen a écrit :
....................................................... |
I think it's a fine idea of it can be toggled in the preferences.Sent from my Android device with K-9 Mail. Please excuse my brevity. |
|
Ok, I will add an option to toggle the gridlines on and off. Originally, I also planned to add alternating background colors to better distinguish between rows. However, since JabRef also allows for marking entries, I doubt that this is a good approach. |
I think you're right about the background colours. It could easily collide with marked entries.Sent from my Android device with K-9 Mail. Please excuse my brevity. |
|
Also I am not a Java programmer (and may be because I am not...), I Frédéric Le 11/05/2015 21:35, Tobias Diez a écrit :
....................................................... |
|
I added an option (under 'Appearance') to re-enable the gridlines. Moreover, it is now possible to change the row height. |
* feat: JabRef#3 add required functionality to bookCoverFetcher (JabRef#8) * feat: add a test for requirement RE02 (JabRef#28) * feat: add a test for requirement RE02 I also had to change the access modifier to `protected` for one download function. Additionally, I refactored the download method a bit, in order to mock at test it more easily. * fix: cleanup imports * feat: JabRef#4 add background download and refresh when opening preview (JabRef#21) * feat: JabRef#34 add missing space and fixed style (JabRef#35) * feat: JabRef#22 add test for requirement RE03 (JabRef#23) * changed comments to markdown (JabRef#44) Co-authored-by: Melker Trané <melkert@kth.se> * feat: JabRef#52 add new test and created setUp and tearDown (JabRef#53) * feat: JabRef#13 add test for retrieving cover from empty directory (JabRef#51) * feat: JabRef#54 add more tests and change modifiers (JabRef#57) * fix: JabRef#43 change variable names to make their type clear (JabRef#59) * made tests more uniform and made originally private methods private agai (JabRef#62) Co-authored-by: Melker Trané <melkert@kth.se> * docs: JabRef#64 added change to changelog (JabRef#66) * almost done * fixed comments * removed line for debugging * fixed typo * added static back * changed !isEmpty to isPresent --------- Co-authored-by: SkitArgDuva <117955752+SkitArgDuva@users.noreply.github.com> Co-authored-by: Melker Trané <melkert@kth.se>
I tried to remove some 'clutter' in the UI by removing the gridlines and a few borders.


Before:
After: