D10429: Disable the title bar separator by default

IlyaBizyaev added a comment. I'll be that one person here who likes the separator and notices it easily on the Breeze theme. I have no objections to changing the default, but "just removing it entirely" is not an option. REPOSITORY R31 Breeze REVISION DETAIL

D15093: Add WireGuard capability.

andersonbruce updated this revision to Diff 41736. andersonbruce added a comment. - Add tests for new and updated ip validators - Update ip validators - Add validator for WireGuard keys - Update per review comments - Add error messages for failure cases REPOSITORY R116 Plasma

D15093: Add WireGuard capability.

andersonbruce marked 45 inline comments as done. andersonbruce added inline comments. INLINE COMMENTS > pino wrote in wireguard.cpp:121 > better use qt's foreach here: > > Q_FOREACH (const QString , valueList) > > and then using `address` instead of `valueList[i]` Used plain 'for' instead

D15550: Move correct indices in the manual sort map when getting a move() call for group children

hein created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. hein requested review of this revision. REVISION SUMMARY Due to not passing `parent` to QAIM::index(), we were moving top-level indices in the map instead. This meant the sort map would become

D10429: Disable the title bar separator by default

IlyaBizyaev added a comment. I think this line's goal is not really to separate the title bar from window content, but instead to make the transition smoother and emphasize style consistency. Take a look at these 2 screenshots, for example. F6267692: Screenshot_20180916_124803.png

D15206: [Kickoff] Add a subtle separator line between the header and the content view

ngraham added a comment. @mmustac, I believe I've addressed your concern now. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D15206 To: ngraham, #plasma, #vdg Cc: abetts, mmustac, hein, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed,

D15206: [Kickoff] Add a subtle separator line between the header and the content view

ngraham updated this revision to Diff 41788. ngraham added a comment. Adjust top margin on Favorites view to make sure that the top item doesn't touch the new line REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15206?vs=40823=41788 BRANCH

D15521: Add validator for lists of IP addressesAdded as separate review per comment from Pino onreview D15093. This code will not compile withoutthe updated code in review D15520. Also includesunit te

jgrulich added inline comments. INLINE COMMENTS > CMakeLists.txt:92 > qca-qt5 > +KF5::NetworkManagerQt > +KF5::Service I'm sure you don't need to add all these and why did you remove PUBLIC and PRIVATE keywords? > CMakeLists.txt:111 > if (WITH_MODEMMANAGER_SUPPORT) > -

D15521: Add validator for lists of IP addressesAdded as separate review per comment from Pino onreview D15093. This code will not compile withoutthe updated code in review D15520. Also includesunit te

jgrulich added a comment. Maybe merge this review with D15520 . I think they should go together. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D15521 To: andersonbruce, jgrulich, pino Cc: plasma-devel,

D15093: Add WireGuard capability.

andersonbruce marked an inline comment as done. andersonbruce added inline comments. INLINE COMMENTS > jgrulich wrote in wireguardadvancedwidget.cpp:104 > I would prefer having just an empty map with data where you just set > everything the user configured in UI, removing options from existing

D15011: [Kickoff] Make the search field always look like a search field

ngraham added a comment. @davidedmundson ping. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D15011 To: ngraham, #plasma, #vdg, davidedmundson, abetts Cc: acrouthamel, fabianr, huftis, rooty, sharvey, romangg, broulik, safaalfulaij, oysteins, filipf,

D15093: Add WireGuard capability.

acrouthamel added a comment. Thank you @andersonbruce for all of your work on this, and quickly working on feedback changes! This will be a great addition. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D15093 To: andersonbruce, #plasma,