- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Feb 25 2020
Feb 24 2020
In D27563#615561, @ngraham wrote:What's the chance we could unify the mobile and desktop versions at some point. Seems a bit anti-Kirigami-ish to maintain a separate mobile view rather than using the power of convergence.
Feb 21 2020
Similar fix applied to plasma-nm. Looks good to me.
Thank you David.
Thank you David.
Feb 18 2020
In D27199#613246, @ngraham wrote:In D27199#612689, @jgrulich wrote:This is probably not related to the UI, it might be issue in the internals. Can you open a bug for it? Ideally if you can restart plasmashell and enable plasma-nm debug
Will do. How do I do enable plasma-nm debug?
Feb 17 2020
Accepting from my side. I'll wait for Nate to do the final review as I don't consider myself expert to QML and UI.
In D27199#611665, @ngraham wrote:In D27199#611308, @gvgeo wrote:Yes, I can still reproduce the issue with this diff applied on top of master. HOWECVER! I just noticed that I can reproduce the issue on master right now, with no patches. I can't believe I missed this before, sorry!
Feb 12 2020
I like it. I didn't go through the code changes yet, I'll wait until you finish it completely.
Looking at some other frameworks, they basically have same code to generate pri files, are they all broken?
Feb 10 2020
Improve cmake file
Fixed all issues.
Improve includes
Drop unrelated change
- Fix cmake variable name
Simplify cmake code
Feb 9 2020
Feb 7 2020
Feb 5 2020
Thinking about it now, what's the reason to put the checkboxes in the plasma-nm applet at the bottom? In this case they overrule what it's visible below, e.g. you turn off WiFi and all wireless networks will disappear. Similar situation is with the notification applet, where the "Do not disturb" checkbox is at the top, because it overrules what's below. It makes more sense when this is placed at the top as it has bigger effect.
Feb 3 2020
In T10470#219347, @gvgeo wrote:
Feb 1 2020
Jan 31 2020
- Add comment why we reverted that change and when it can be removed
Jan 29 2020
Jan 23 2020
Jan 21 2020
Jan 16 2020
Jan 15 2020
- Code improvements
Jan 13 2020
Address review comments:
- Add text to the toolbutton and make it checkable
- Allow to create hotspot only if WiFi is available or it's not used as primary connection
In D26392#592288, @ngraham wrote:In D26392#592133, @jgrulich wrote:Can someone please look into this review? Either try it or check the code? I would like to have this as part of Plasma 5.18 and deadline for that is in few days. @ngraham how is it going with the icon?
@cblack is working on it IIRC. It's too late to show the icon in Plasma 5.18 anyway (the icon will be in the breeze-icons repo which is a framework, and Frameworks 5.66 which Plasma 5.18 relies on has already been tagged. So there's no huge rush there IMO.
I'm still unable to test this because of the odd build failure that I cannot understand, explain, or overcome. :/
In D26392#592136, @apol wrote:In D26392#588345, @jgrulich wrote:In D26392#587159, @mthw wrote:@apol You are right, it is not possible to create a hotspot if one is already connected to a WiFi network. Currently enabling hotspot disables your previous connection (WiFi) and hides available WiFi networks.
That's a question. Do we want to allow to create a hotspot if users are already connected to w WiFi network? You are allowed to do this for example on Android. Users might not realize why they are not allowed to create a hotspot and disconnecting from WiFi doesn't seem to me be a good first step.
Leaving the user without a connection would be quite dishearting and possibly confusing.
I can imagine the use-case of: I am in this building with weird authentication (thinking university) and I want to share that connection with my phone, so I create a hotspot and we don't have to do weird configuration on the phone. Would apply to e.g. vpns too.
Jan 11 2020
Can someone please look into this review? Either try it or check the code? I would like to have this as part of Plasma 5.18 and deadline for that is in few days. @ngraham how is it going with the icon?
Jan 9 2020
In D26392#590884, @ngraham wrote:No, even with networkmanager-qt from git master, I still get the same build failure here. :(
In D26392#588994, @ngraham wrote:This no longer compiles for me:
/home/nate/kde/src/plasma-nm/libs/handler.cpp: In member function ‘void Handler::createHotspot()’: /home/nate/kde/src/plasma-nm/libs/handler.cpp:622:94: error: ‘addAndActivateConnection2’ is not a member of ‘NetworkManager’; did you mean ‘addAndActivateConnection’? 622 | QDBusPendingReply<QDBusObjectPath, QDBusObjectPath, QVariantMap> reply = NetworkManager::addAndActivateConnection2(connectionSettings->toMap(), wifiDev->uni(), QString(), options); | ^~~~~~~~~~~~~~~~~~~~~~~~~ | addAndActivateConnection
Jan 7 2020
I don't really remember, but I can see in the userHash() it's not used so this is probably safe. In the userHash() we directly use KUser().
Jan 6 2020
It requires networkmanager-qt from master (upcoming 5.66 version).
I used ToolButton instead of a regular button, reason is that I don't think there is enough space. If there is modem device available, there will be three checkboxes on the left and with a regular button in various languages this might not be enough space.
Use button instead of checkbox
In D26392#587200, @alexde wrote:UI-wise, this would probably be better off as a button than a checkbox.
Personally, I'm more inclined to a checkbox. Why is "turn wifi on/off" not an action? Right now, I don't see the big difference.
Is the new "hotspot" equivalent to the "shared wifi", you can already set up?
If so, without really reading the patch details yet, just some remarks or ideas:
- If you have configured several hotspots in the KCM, you probably need to choose a default one or a specific one when activating the hotspot.
In D26392#587159, @mthw wrote:@apol You are right, it is not possible to create a hotspot if one is already connected to a WiFi network. Currently enabling hotspot disables your previous connection (WiFi) and hides available WiFi networks.
Jan 3 2020
There are two things which need work or I'm not sure about:
- If this should be placed as a checkbox in the applet
- We need a different icon for the hotspot, currently there is no such icon in the Plasma theme and freeze for Plasma 5.18 is soon. This is meant to be a kind request for someone who does icons for Plasma.
Jan 2 2020
Dec 20 2019
Dec 18 2019
I will land this once Flatpak 1.6 is released. While this can be used by older Flatpak version, it will not build with older Flatpak versions.
Simplify tooltips