antlarr created this revision.
antlarr added a reviewer: Aiix.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
antlarr requested review of this revision.
REVISION SUMMARY
The skills folder was hardcoded to /opt/mycroft/skills in
antlarr created this revision.
antlarr added a reviewer: Aiix.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
antlarr requested review of this revision.
REVISION SUMMARY
Each time a message is received, the focus was stolen from
the
antlarr created this revision.
antlarr added a reviewer: broulik.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
antlarr requested review of this revision.
REVISION SUMMARY
The chromium extension wasn't working when LANG was set to
antlarr updated this revision to Diff 20239.
antlarr added a comment.
- Remove check for currentPort which is no longer needed
REPOSITORY
R115 Plasma Audio Volume Applet
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D8054?vs=20094=20239
BRANCH
master
REVISION DETAIL
antlarr added a comment.
Thanks for the hint! I didn't know I could use FormFactor directly to
reference the Device's formFactor property. Indeed now it looks much better.
REPOSITORY
R115 Plasma Audio Volume Applet
REVISION DETAIL
https://phabricator.kde.org/D8054
To: antlarr, #plasma,
antlarr marked an inline comment as done.
REPOSITORY
R115 Plasma Audio Volume Applet
REVISION DETAIL
https://phabricator.kde.org/D8054
To: antlarr, #plasma, davidedmundson, drosca
Cc: drosca, sebas, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,
jensreuterberg, abetts, apol,
antlarr updated this revision to Diff 20094.
antlarr added a comment.
- Use the formFactor attribute of the Device directly
REPOSITORY
R115 Plasma Audio Volume Applet
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D8054?vs=20090=20094
BRANCH
master
REVISION DETAIL
antlarr updated this revision to Diff 20090.
antlarr added a comment.
- Use form_factor from device to select the icon for a profile
REPOSITORY
R115 Plasma Audio Volume Applet
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D8054?vs=20082=20090
BRANCH
master
REVISION DETAIL
antlarr added a comment.
I wrote in the description the contents of the string variables we could work
with so that it's clear I couldn't find any other way to extract that
information. If you have any other idea (any other variable to use for the
match?) I'm happy to test it.
REPOSITORY
antlarr added a comment.
Perfect, thanks, I'll do that
REPOSITORY
R115 Plasma Audio Volume Applet
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D8054
To: antlarr, #plasma, davidedmundson
Cc: sebas, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,
antlarr added a comment.
If I move it to the end, then the player device will show a headset correctly
anyway since no other condition matches but for the capture device, a
microphone icon will be shown instead of the headset.
REPOSITORY
R115 Plasma Audio Volume Applet
BRANCH
master
antlarr added a comment.
Hmm, you're right, there's no audio-headset icon in breeze, it's inherited
from oxygen. Should we use it while we request an audio-headset icon for
breeze? or should I revert that change and use headphones in the meantime?
REPOSITORY
R115 Plasma Audio Volume
antlarr retitled this revision from "Show headphone icons for devices
identifying as "Headset" in Description" to "Show headset icons for devices
identifying as "Headset" in Description".
antlarr edited the summary of this revision.
antlarr edited the test plan for this revision.
REPOSITORY
antlarr updated this revision to Diff 20082.
antlarr added a comment.
- We have a headset icon, so use it for headsets
REPOSITORY
R115 Plasma Audio Volume Applet
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D8054?vs=20078=20082
BRANCH
master
REVISION DETAIL
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
A Logitech H600 / H800 headset was showing an "unknown" icon.
These devices use the following
This revision was automatically updated to reflect the committed changes.
Closed by commit rPLASMAWORKSPACE18858d16d6b1: Modified distance function to
keep aspect ratio as close as possible (authored by antlarr).
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
CHANGES SINCE LAST UPDATE
antlarr updated this revision to Diff 7823.
antlarr added a comment.
- Moved testfindpreferredimage to an autotests directory
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3188?vs=7764=7823
BRANCH
arcpatch-D3188
REVISION DETAIL
antlarr added a comment.
@hpereiradacosta thanks for reviewing this. Can you cherry-pick the commit
for Plasma/5.8?
REPOSITORY
rBREEZE Breeze
REVISION DETAIL
https://phabricator.kde.org/D3192
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr,
antlarr closed this revision.
REPOSITORY
rBREEZE Breeze
REVISION DETAIL
https://phabricator.kde.org/D3192
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #plasma, #plasma:_design, hpereiradacosta
Cc: hpereiradacosta, davidedmundson,
antlarr updated this revision to Diff 7788.
antlarr added a comment.
- Increase width by MenuButton_IndicatorWidth+2 instead of adding a new enum
entry
REPOSITORY
rBREEZE Breeze
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3192?vs=7735=7788
BRANCH
master
REVISION DETAIL
antlarr added inline comments.
INLINE COMMENTS
> davidedmundson wrote in breezestyle.cpp:2507
> you're still basing the height off the MenuButton_IndicatorWidth which you're
> otherwise not using
>
> Is that intentional?
Good point, I didn't notice that it was using MenuButton_IndicatorWidth
antlarr updated this revision to Diff 7764.
antlarr added a comment.
- Adjust the distance method to work with QUAD HD resolutions
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3188?vs=7737=7764
BRANCH
Plasma/5.8
REVISION DETAIL
antlarr marked 3 inline comments as done.
antlarr added a comment.
Converted the test to use QTest
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D3188
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To:
antlarr updated this revision to Diff 7737.
antlarr added a comment.
- Converted the test to use QTest
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3188?vs=7719=7737
BRANCH
Plasma/5.8
REVISION DETAIL
antlarr added a reviewer: Plasma: Design.
antlarr added a project: Plasma: Design.
REPOSITORY
rBREEZE Breeze
REVISION DETAIL
https://phabricator.kde.org/D3192
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #plasma, #plasma:_design
Cc:
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
When the size of the combo box adjusts to its contents, the dropdown
listbox isn't large enough to fit the
antlarr added inline comments.
INLINE COMMENTS
> mart wrote in image.cpp:196
> dead code
The commented out qDebug a couple of lines below shows that variable (when
uncommented). Should I remove it from the qDebug line below? or just keep both?
(the same for the case above)
> mart wrote in
antlarr added inline comments.
INLINE COMMENTS
> testfindpreferredimage.cpp:91
> + if (preferred != expected) failed++;
> + std::cout << "For a screen size of " <<
> qPrintable(formatResolution(resolution))
> + << " the " << qPrintable(formatResolution(preferred)) << "
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
This new version of the distance function simplifies the
findPreferredImageInPackage method by using the
antlarr closed this revision.
REPOSITORY
rKWIN KWin
REVISION DETAIL
https://phabricator.kde.org/D3132
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, davidedmundson, #plasma, #kwin, graesslin
Cc: luebking, graesslin, kwin, plasma-devel,
antlarr added a comment.
I don't know why, but it seems arc diff doesn't work anymore and I have to
use arc diff HEAD~7 to include all 7 commits
REPOSITORY
rKWIN KWin
REVISION DETAIL
https://phabricator.kde.org/D3132
EMAIL PREFERENCES
antlarr updated this revision to Diff 7644.
antlarr added a comment.
Restricted Application edited projects, added Plasma; removed KWin.
Include all 7 commits
REPOSITORY
rKWIN KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3132?vs=7638=7644
BRANCH
arcpatch-D3127_1
antlarr marked 5 inline comments as done.
antlarr added a comment.
All issues done
REVISION DETAIL
https://phabricator.kde.org/D3132
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #kwin, #plasma, davidedmundson
Cc: luebking, graesslin, kwin,
antlarr updated this revision to Diff 7638.
antlarr added a comment.
Restricted Application edited projects, added KWin; removed Plasma.
- Only call PostLastGuardedFrame when OpenGLCompositing is set
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3132?vs=7637=7638
BRANCH
antlarr added inline comments.
INLINE COMMENTS
> luebking wrote in x11_platform.cpp:206
> PreFrame is (now) effectively "PreFirstGuardedFrame", is it?
> And if invoked at some later point would create the timer and hit the config
> rewrite every single frame (for the counter is stuck at 0)?
>
antlarr updated this revision to Diff 7637.
antlarr added a comment.
Restricted Application edited projects, added Plasma; removed KWin.
Include all 6 commits
REPOSITORY
rKWIN KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3132?vs=7636=7637
BRANCH
arcpatch-D3127_1
antlarr updated this revision to Diff 7636.
antlarr marked 21 inline comments as done.
antlarr added a comment.
Restricted Application edited projects, added KWin; removed Plasma.
- Use only one timer to detect freezes and other optimizations
- Test 3 frames for freezes by default
CHANGES
antlarr closed this revision.
REPOSITORY
rUSERMANAGER User Manager
REVISION DETAIL
https://phabricator.kde.org/D3102
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #plasma, davidedmundson
Cc: mart, plasma-devel, lesliezhai, ali-mohamed,
antlarr added inline comments.
INLINE COMMENTS
> graesslin wrote in composite.h:242
> I would just count down the m_maxFramesTestedForSafety till it reaches 0
ok
> graesslin wrote in x11_platform.cpp:215
> You can use QMetaObject::invokeMethod with Qt::QueuedConnection.
Oh, I thought
antlarr added inline comments.
INLINE COMMENTS
> luebking wrote in x11_platform.cpp:215
> You should be able to call it as slot (whether QMetaObject::invokeMethod()
> works, I've never tried)
yeah, but then it wouldn't be a synchronous call and it would lose all its
meaning, isn't it? :)
>
antlarr added inline comments.
INLINE COMMENTS
> luebking wrote in composite.h:242
> m_testedFrames or similar, "Frame Count" is too generic.
> Alternatively, just use some "int m_freezeDectionFrames = 30;" and count
> them down (so when < 1, you're done)
ok. @graesslin do you prefer a count
antlarr added a comment.
Btw, I tested the last commit on a virtual machine simulating the freeze with
a sleep. I'll check tomorrow what's the minimum value for
m_maxFramesTestedForSafety that works to detect the real freeze and update its
default value.
REVISION DETAIL
antlarr updated this revision to Diff 7613.
antlarr added a comment.
Restricted Application edited projects, added Plasma; removed KWin.
- Remove most overhead caused by the freeze-detection thread
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3132?vs=7599=7613
BRANCH
antlarr added a comment.
> How many frames do we need to detect the freeze? I think 30 frames is too
much by default. It's adding quite some stress on startup (lots of write to
config, lots of additional threads started). Assuming we freeze with tripple
buffer when trying to get the next
antlarr added a comment.
I tested this patch but it didn't work at detecting the freeze. Fortunately,
after debugging kwin for a bit I worked on this and made a version that detects
the freezes correctly.
I think I cannot add commits here, so I created a new phab at
antlarr created this revision.
antlarr added reviewers: KWin, Plasma, davidedmundson.
antlarr added subscribers: davidedmundson, plasma-devel, kwin.
antlarr added a project: Plasma.
Restricted Application edited projects, added KWin; removed Plasma.
REVISION SUMMARY
With nouveau driver it can
antlarr closed this revision.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D3128
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #plasma, davidedmundson
Cc: plasma-devel, lesliezhai, ali-mohamed,
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
If the user list is empty (for example, because regular users are hidden
or sddm's configuration includes
antlarr closed this revision.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D2981
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #plasma, mart
Cc: mart, plasma-devel, lesliezhai, ali-mohamed,
antlarr added a comment.
Oops, sorry, I wrote a reply yesterday but forgot to submit it :/
INLINE COMMENTS
> mart wrote in accountmodel.cpp:165
> is there absolutely nothing left needing this?
I can't find anything at least. I tested the systemsettings window is updated
correctly
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Only save values that were really changed
This allows to change settings without being asked for the
antlarr added inline comments.
INLINE COMMENTS
> mart wrote in image.cpp:245
> shouldn't then "preferred" just not be defined anymore in the wallpaper
> package structure then?
Sorry, I don't get your comment. That's why I remove it, so that "preferred" is
not defined anymore and the next
too.
(1) The plasma patch is at https://phabricator.kde.org/D2981
Thanks,
Antonio Larrosa Jimenez
d->discoveries too.
(1) The plasma patch is at https://phabricator.kde.org/D2981
Thanks,
Antonio Larrosa Jimenez
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Removes preferred definition so a new wallpaper is selected.
Also, just in case, check for division by 0.
ath("preferred") is not empty at the beginning of the method
even if package.removeDefinition("preferred") was just called, so it seems
the key should be removed from d->discoveries too.
(1) Plasma patch is comming to phabricator in the next minutes.
Thanks,
Antonio Larrosa Jimenez
This revision was automatically updated to reflect the committed changes.
Closed by commit rPLASMAWORKSPACE189546c2696a: Renamed userPrompt to
userPromptComponent and add userListComponent id (authored by antlarr).
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
CHANGES SINCE LAST UPDATE
antlarr updated this revision to Diff 6968.
antlarr added a comment.
- Move logic to show/hide avatars from components/SessionManagementScreen.qml
to Main.qml
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D2874?vs=6954=6968
BRANCH
antlarr added a comment.
Thanks for your comments. I should warn you that this is the second time I
touch qml code, so apologies in advance :).
INLINE COMMENTS
> davidedmundson wrote in SessionManagementScreen.qml:60
> this component is shared with the lockscreen and user switcher, you're
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Renamed userPrompt to userPromptComponent and add an id
(userListComponent) to the initialItem Login
antlarr closed this revision.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D2834
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antlarr, #plasma, davidedmundson
Cc: plasma-devel, lesliezhai, ali-mohamed,
antlarr created this revision.
antlarr added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
The default wallpaper method is to scale the wallpaper, and the previous
algorithm in plasma choosed the
view87307
---
On Oct. 23, 2015, 1:06 p.m., Antonio Larrosa Jimenez wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewb
main.cpp 42f4c6a527e0ba6a76529d1f25bfe1e5d9a485bf
Diff: https://git.reviewboard.kde.org/r/125764/diff/
Testing
---
Noticed that "kinfocenter --version" crashed.
I made this patch, built it and tested it doesn't crash anymore.
Thanks,
Antonio Larrosa Jimenez
oesn't crash anymore.
Thanks,
Antonio Larrosa Jimenez
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
65 matches
Mail list logo