D28024: [Baloo KCM] Expose "Index hidden" config setting

2020-03-15 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R119:491ce0f1d78f: [Baloo KCM] Expose Index hidden config setting (authored by bruns). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28024?vs=77558=77662

D28024: [Baloo KCM] Expose "Index hidden" config setting

2020-03-14 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. All right, let's go for it as-is. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28024 To: bruns, #baloo, #vdg, ngraham Cc: plasma-devel, Orage,

D28024: [Baloo KCM] Expose "Index hidden" config setting

2020-03-14 Thread Stefan Brüns
bruns added inline comments. INLINE COMMENTS > ngraham wrote in main.qml:59 > Instead adding these new layouts, you could just set the following on each > checkbox that needs to be indented: > > Layout.leftMargin: !LayoutMirroring.enabled ? Kirigami.Units.largeSpacing : > 0 >

D28024: [Baloo KCM] Expose "Index hidden" config setting

2020-03-14 Thread Nathaniel Graham
ngraham added inline comments. INLINE COMMENTS > main.qml:59 > + > +ColumnLayout { > +QQC2.CheckBox { Instead adding these new layouts, you could just set the following on each checkbox that needs to be indented: Layout.leftMargin: !LayoutMirroring.enabled ?

D28024: [Baloo KCM] Expose "Index hidden" config setting

2020-03-13 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, VDG, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Also inset both "Index file content" and "Index hidden" so it is more obvious both are