mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Remove lower row height in virtualized table on macOS

Browse source 
With the additional PDF icon in the Attachments column from b0ba7e464,
might as well have the same amount of spacing to tell the rows apart.
This commit is contained in:
Dan Stillman 2021-12-22 03:42:15 -05:00
parent b0ba7e4646
commit 0f676dd720
2 changed files with 0 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chrome/content/zotero/collectionTree.jsx
View file

@ -344,8 +344,6 @@ var CollectionTree = class CollectionTree extends LibraryTree {
ref: ref => this.tree = ref,
treeboxRef: ref => this._treebox = ref,
renderItem: this.renderItem,
// Extra space on macOS (which gets reduced by 2, so this really means 20)
...(Zotero.isMac && { rowHeight: 22 }),
alternatingRowColors: null,
onSelectionChange: this._handleSelectionChange,

7
chrome/content/zotero/components/virtualized-table.jsx
View file

@ -297,10 +297,6 @@ class VirtualizedTable extends React.Component {
if (!props.disableFontSizeScaling) {
this._rowHeight *= Zotero.Prefs.get('fontSize');
}
// A bit less row spacing on macOS
if (Zotero.isMac && this._rowHeight >= (props.rowHeight || DEFAULT_ROW_HEIGHT)) {
this._rowHeight -= 2;
}
this.selection = new TreeSelection(this);
@ -1156,9 +1152,6 @@ class VirtualizedTable extends React.Component {
}
this._rowHeight = this.props.rowHeight || DEFAULT_ROW_HEIGHT;
this._rowHeight *= Zotero.Prefs.get('fontSize');
if (Zotero.isMac && this._rowHeight >= (this.props.rowHeight || DEFAULT_ROW_HEIGHT)) {
this._rowHeight -= 2;
}
if (!this._jsWindow) return;
this._jsWindow.update(this._getWindowedListOptions());