Remove the padding of the bookmark bar
Needs ReviewPublic

Authored by trmdi on Fri, Apr 26, 4:05 AM.

Details

Reviewers
None
Group Reviewers
Falkon
VDG
Summary

Remove the padding of the bookmark bar

Test Plan

Breeze widget style:


Fusion widget style:

Diff Detail

Repository
R875 Falkon
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11304
Build 11322: arc lint + arc unit
trmdi created this revision.Fri, Apr 26, 4:05 AM
Restricted Application added a project: Falkon. · View Herald TranscriptFri, Apr 26, 4:05 AM
Restricted Application added a subscriber: falkon. · View Herald Transcript
trmdi requested review of this revision.Fri, Apr 26, 4:05 AM
trmdi edited the test plan for this revision. (Show Details)Fri, Apr 26, 4:06 AM
ngraham accepted this revision as: VDG.Fri, Apr 26, 3:18 PM
ngraham added a subscriber: ngraham.

+1 for UI. Let's see if the Falkon folks think this is the right way to implement the change.

drosca added a subscriber: drosca.Sat, Apr 27, 5:38 AM

Did you test with other Qt styles (fusion, windows, ...) ?

trmdi updated this revision to Diff 57113.Sat, Apr 27, 5:11 PM

Remove the padding of the bookmark bar

trmdi edited the summary of this revision. (Show Details)Sat, Apr 27, 5:20 PM
trmdi edited the test plan for this revision. (Show Details)
trmdi retitled this revision from Remove the padding of BookmarksToolbarButton to Remove the padding of the bookmark bar.

Did you test with other Qt styles (fusion, windows, ...) ?

Yes, it works too.