https://bugs.kde.org/show_bug.cgi?id=374311
--- Comment #14 from Christoph Feck ---
Qt developers say it is fixed in Qt 6.
--
You are receiving this mail because:
You are the assignee for the bug.
On 12/08/20 16:23, Wang Gary wrote:
An alternate calendar is a feature to allow another calendar system's date
name display together with the main calendar's date names.
Is the calendaring system needed for China supported in Qt? Please see
https://doc.qt.io/qt-5/qcalendar.html#System-enum
Thi
https://bugs.kde.org/show_bug.cgi?id=374311
Christoph Feck changed:
What|Removed |Added
CC||krzmb...@gmail.com
--- Comment #12 from
On 10/16/20 15:26, Ilya Bizyaev wrote:
I've had a wild idea of doing a KDE-related engineering project
Thank you for your interest in helping the KDE community!
If you exclude anything reported to bugs.kde.org (which does not only
list bugs, but also has ideas for new features), you could als
https://bugs.kde.org/show_bug.cgi?id=424128
Christoph Feck changed:
What|Removed |Added
Status|REPORTED|RESOLVED
Resolution
https://bugs.kde.org/show_bug.cgi?id=374311
Christoph Feck changed:
What|Removed |Added
CC||oceanodefol...@pm.me
--- Comment #8 from
cfeck added a comment.
Instead of darker, you could use the text color with very small alpha.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D29081
To: ngraham, #vdg, #breeze, niccolove, ndavis
Cc: cfeck, ndavis, niccolove, broulik, plasma-devel, Orage, LeGast00n,
The
cfeck added a comment.
What a small difference that makes such a large difference!
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D29081
To: ngraham, #vdg, #breeze, niccolove, ndavis
Cc: cfeck, ndavis, niccolove, broulik, plasma-devel, Orage, LeGast00n,
The-Feren-OS-D
cfeck added a comment.
It looks like the submitter doesn't have commit rights.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D28941
To: auxym, #plasma, davidedmundson, apol
Cc: cfeck, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh,
zach
cfeck added a comment.
Anyone else able to review this?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D25428
To: steffenh, #plasma, hein, ngraham, mart
Cc: cfeck, mart, ngraham, nicolasfella, plasma-devel, Orage, LeGast00n,
The-Feren-OS-Dev, cblack, jraleigh
cfeck added a comment.
See https://bugreports.qt.io/browse/QTBUG-56547
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D27808
To: ahmadsamir, #plasma, davidedmundson, broulik, ervin, meven
Cc: cfeck, bport, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cbla
cfeck added a comment.
We already lost fractional point sizes when porting from KFontDialog to
QFontDialog. Best approach is to add the KFontDialog to the platform plugin
(and also the old KColorDialog while you are at it), or upstream our features
to Qt.
Btw, I set Toolbar font to a "c
cfeck added a comment.
If this is still needed, please commit and clarify the status of the
referenced bug.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D27650
To: alnikiforov, ngraham, davidedmundson, mart
Cc: cfeck, anthonyfieroni, plasma-devel, Orage,
release scrips and others will get a surprise when
they want to make contributions to it.
We have https://phabricator.kde.org/T11994 to track those moves, please
subscribe there.
--
Christoph Feck
KDE Release Team
cfeck added a comment.
So... is this all related to bug https://bugs.kde.org/show_bug.cgi?id=378523 ?
:/
REPOSITORY
R99 KDE Gtk Configuration Tool
REVISION DETAIL
https://phabricator.kde.org/D27380
To: gikari, #plasma, ervin, bport, meven
Cc: cfeck, ngraham, IlyaBizyaev, plasma-devel,
cfeck added a comment.
Since this patch isn't critical, it would make a perfect candidate for your
first steps with committing :)
REPOSITORY
R110 KScreen Library
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D27383
To: z3ntu, davidedmundson
Cc: cfeck, romangg, plasma-deve
cfeck added a comment.
Oh, okey. That was just misleading.
I also suggest to ask for commit rights then.
REPOSITORY
R110 KScreen Library
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D27383
To: z3ntu, davidedmundson
Cc: cfeck, romangg, plasma-devel, Orage, LeGast00n,
cfeck added a comment.
According to
https://phabricator.kde.org/R982:8dffcc64d2aec4041129d67809839f136fec080a you
have commit rights?
REPOSITORY
R110 KScreen Library
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D27383
To: z3ntu, davidedmundson
Cc: cfeck, romangg, plasma
cfeck added a comment.
That looks clever. Since CpuInfoBufSize is a static variable, it will
remember the last used size and not reallocate again (unless you plug in new
CPUs).
Is realloc() an option? It should help the case where the library can
actually grow the buffer without an allo
cfeck added a comment.
From that bug report:
Newer x86_64 kernels, however, allow up to 4096 CPUs, so either the buffer
needs to be much bigger, or we should do interleaved /proc/cpuinfo reading and
parsing.
REPOSITORY
R106 KSysguard
REVISION DETAIL
https://phabricator.kde.org/D273
On 02/13/20 08:42, Ben Cooksley wrote:
Part of the issue here is that Plasma has been known to add API to
Frameworks and then immediately, without any delay, start using it
(pretty much always breaking CI in the process)
This means that other changes are likely being pushed into Frameworks
by Pl
cfeck added a comment.
But the timeout isn't used to give apps more time needed to clean up before
they confirm the session can be closed? If I understand it correctly, there
should be two timeouts?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D27077
To:
cfeck added a comment.
Why is there a timeout at all?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D27077
To: ngraham, #vdg, #plasma
Cc: cfeck, davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev,
jraleigh, zachus, fbampaloukas, GB_2, ragreen
cfeck added inline comments.
INLINE COMMENTS
> standard_actions_module.cpp:73
> +m_editor->setMinimumWidth(700);
> +m_editor->setMinimumHeight(600);
> +
`setMinimumSize()`
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D26695
To: ngraham, #plasma
Cc: cf
cfeck added a comment.
Note that a window's minimum size does not include decorations or the panel.
I suggest to subtract 60 pixels from the minimum height, otherwise on a 800x600
screen the buttons are likely hidden behind the panel.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https
cfeck added a comment.
If application developers need a rotated button, they need to rotate only the
contents, not the frame.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D24706
To: ndavis, #vdg, #breeze
Cc: cfeck, The-Feren-OS-Dev, cblack, bodoeggert, ngraham, plas
cfeck added a comment.
The radio stream I use (via mpv) shows the interpret and title information
for all titles it plays. Also, when mpv plays a local file, ttitle information
about the file is added there. I don't use any other media streams, so I cannot
comment on the general availability
https://bugs.kde.org/show_bug.cgi?id=369158
Christoph Feck changed:
What|Removed |Added
Status|NEEDSINFO |RESOLVED
Resolution|WAITINGFORINFO
https://bugs.kde.org/show_bug.cgi?id=375909
Christoph Feck changed:
What|Removed |Added
Status|NEEDSINFO |RESOLVED
Resolution|WAITINGFORINFO
cfeck added a comment.
Shouldn't there be a warning message "delete yes/no?"
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D25280
To: ngraham, tcanabrava, #plasma, davidedmundson
Cc: cfeck, davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh,
cfeck added a comment.
Please make any GTK stuff optional, not required.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D21860
To: gikari, #plasma, apol, davidedmundson
Cc: cfeck, davidedmundson, nicolasfella, broulik, GB_2, abetts, ngraham,
plasma-devel, LeG
On 09/14/19 14:38, Carl Schwan wrote:
I finalized the first version of the new plasma-desktop webpage, a preview is
available at https://kde.carlschwan.eu/plasma-desktop. There are still two
parts missing (p-b-i and privacy) and some possible improvement that I would
like to add later. Can I g
in case of compilation errors.
--
Christoph Feck
KDE Release Team
cfeck added a comment.
Isn't Qt Quick amazing?
REPOSITORY
R119 Plasma Desktop
BRANCH
add-subtle-icon-shadow-to-folderview (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D23803
To: ngraham, hein, #plasma, #vdg, filipf, ndavis
Cc: cfeck, ndavis, filipf, plasma-devel,
cfeck added a comment.
I believe 1.25 is a common scale factor. Is something else in the stack
limited to one digit?
REPOSITORY
R104 KScreen
REVISION DETAIL
https://phabricator.kde.org/D23732
To: romangg, #plasma
Cc: cfeck, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampalouk
cfeck added a comment.
Could someone test this on Wayland? The fix itself looks trivial, but if it
breaks Wayland session, it obviously needs to be changed.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D22740
To: marten, #plasma
Cc: cfeck, plasma-devel, L
This revision was automatically updated to reflect the committed changes.
Closed by commit R102:8056039c7947: Fix PCI module (authored by konradr,
committed by cfeck).
REPOSITORY
R102 KInfoCenter
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D18920?vs=61562&id=62955
REVISION DETAIL
This revision was automatically updated to reflect the committed changes.
Closed by commit R102:ed789f56fc69: Update and add some device names in PCI
module (authored by konradr, committed by cfeck).
REPOSITORY
R102 KInfoCenter
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D22369?vs=
https://bugs.kde.org/show_bug.cgi?id=374311
Christoph Feck changed:
What|Removed |Added
CC||marklikesvideogames@gmail.c
cfeck added a comment.
+1 for putting it in the Help menu.
REPOSITORY
R124 System Settings
REVISION DETAIL
https://phabricator.kde.org/D22728
To: ngraham, #plasma, #vdg
Cc: cfeck, mthw, mmustac, broulik, plasma-devel, LeGast00n, jraleigh,
fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, him
cfeck added a comment.
Do we need a matching commit in KWidgetsAddons? See
https://commits.kde.org/kwidgetsaddons/2631be903f947872f67c9e2063950345dda9033e
REPOSITORY
R858 Qt Quick Controls 2: Desktop Style
REVISION DETAIL
https://phabricator.kde.org/D22630
To: ngraham, #vdg, filipf, cul
cfeck added a comment.
Oh, if the latter syntax also works, then you are right.
REPOSITORY
R126 KDE CLI Utilities
REVISION DETAIL
https://phabricator.kde.org/D22525
To: arrowd
Cc: cfeck, plasma-devel, #frameworks, LeGast00n, jraleigh, fbampaloukas, GB_2,
ragreen, Pitel, ZrenBot, himcesj
cfeck added a comment.
Could we only apply the `:xx` check on the filename part, not on the server
part? Someone might expect that `http://path.to/file.txt:99` also works for
remote files.
REPOSITORY
R126 KDE CLI Utilities
REVISION DETAIL
https://phabricator.kde.org/D22525
To: arrowd
C
This revision was automatically updated to reflect the committed changes.
Closed by commit R102:196d4aaa9cf6: Formatting cleanup in PCI module (authored
by konradr, committed by cfeck).
REPOSITORY
R102 KInfoCenter
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D22368?vs=61497&id=61533
cfeck added inline comments.
INLINE COMMENTS
> kpci_private.h:48
>
> +/* Following structurs MUST BE packed, otherwise variables will be mapped to
> + wrong places. Q_PACKED macro seems not work very well. `#pragma pack` are
structure or structures?
> kpci_private.h:49
> +/* Following struc
This revision was automatically updated to reflect the committed changes.
Closed by commit R102:96c18ee3cf63: Update Konrad's mail address in PCI
module (authored by cfeck).
REPOSITORY
R102 KInfoCenter
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D22367?vs=61496&id=61531
REVISION D
This revision was automatically updated to reflect the committed changes.
Closed by commit R135:d2a1d1ca9e9e: [KDEPlatformSystemTrayIcon] Set
KStatusNotifierItem to Active (authored by cfeck).
REPOSITORY
R135 Integration for Qt applications in Plasma
CHANGES SINCE LAST UPDATE
https://phabric
cfeck created this revision.
cfeck added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
cfeck requested review of this revision.
REVISION SUMMARY
Qt applications expect that QSystemTrayIcon's default behavior is to show the
icon.
BUG: 407963
R
cfeck added a comment.
See also bug 403953 and bug 408863.
REPOSITORY
R123 SDDM Configuration Panel (KCM)
REVISION DETAIL
https://phabricator.kde.org/D22191
To: filipf, #plasma, ngraham, davidedmundson, #vdg
Cc: cfeck, GB_2, ndavis, plasma-devel, LeGast00n, jraleigh, fbampaloukas,
ragre
cfeck added a comment.
Would it be possible to commit the whitespace changes separately? They hide
all the actual changes.
REPOSITORY
R102 KInfoCenter
REVISION DETAIL
https://phabricator.kde.org/D18920
To: konradr, #plasma, cfeck
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2
cfeck added a comment.
Is this fixed by https://phabricator.kde.org/D21894 ?
REPOSITORY
R871 DrKonqi
REVISION DETAIL
https://phabricator.kde.org/D19855
To: marcelomariano, sitter, cfeck, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel,
ZrenBot, himcesjf
cfeck accepted this revision.
cfeck added a comment.
This revision is now accepted and ready to land.
Thanks!
REPOSITORY
R111 KSysguard Library
REVISION DETAIL
https://phabricator.kde.org/D21828
To: kishoreg, cfeck
Cc: cfeck, plasma-devel, LeGast00n, jraleigh, GB_2, ragreen, Pitel, ZrenB
cfeck added a comment.
If the backtrace is filled later, and that would cause the minimumSize to
change, then it will automatically get propagated to the window.
A wordwrapped label is supposed to adapt to any with _or_ any height, so its
minimum size is its minimum with and its minimum
cfeck added a comment.
I guess sizeHint() from a paged dialog doesn't pick all pages, so the correct
way to resolve this issue is to iterate over all pages, compute their
sizeHint(), and use the combined size as the minium size, see
`QSize::expandedTo()`.
REPOSITORY
R871 DrKonqi
REVISION
cfeck added a comment.
I think I now see why we messed with sizes. There are wrapped QLabels in the
dialog, and these break minimum size hints.
What we generally do to resolve this issue is to use
`setMinimumSize(sizeHint())`.
REPOSITORY
R871 DrKonqi
REVISION DETAIL
https://phabric
cfeck added a comment.
What happens if you (instead/also) remove the next line?
REPOSITORY
R871 DrKonqi
REVISION DETAIL
https://phabricator.kde.org/D21797
To: sitter, #plasma
Cc: cfeck, plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel,
ZrenBot, himcesjf, lesliezhai, al
cfeck added a comment.
There will be no further 5.15 release. Please set to 5.16 branch (and merge
to master when ready).
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D21646
To: sebas, #plasma
Cc: cfeck, broulik, zzag, plasma-devel, LeGast00n, ericadams, jraleigh, G
https://bugs.kde.org/show_bug.cgi?id=304696
Christoph Feck changed:
What|Removed |Added
CC||yst...@posteo.net
--- Comment #15 from
cfeck added a comment.
Just an idea: run "kcharselect", type "circled digit" resp. "circled number".
For values <= 20, unicode has pre-rendered glyphs.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D21408
To: broulik, #plasma, #vdg
Cc: cfeck, davidedmundso
cfeck added inline comments.
INLINE COMMENTS
> NotificationItem.qml:74
>
> -property int headingLeftPadding: 0
> +property int headingLeftPadding: units.smallSpacing
> property int headingRightPadding: 0
Does this take into account layouts for RTL languages?
REPOSITORY
R120 Pla
cfeck requested changes to this revision.
cfeck added a comment.
This revision now requires changes to proceed.
And indeed I didn't mark the bug as a junior job, because incremental parsing
the info isn't exactly a quick fix.
REPOSITORY
R106 KSysguard
REVISION DETAIL
https://phabricator.
cfeck added a comment.
The info for each CPU is indeed a bit larger than 1K. Using 64KiB will only
be sufficient for 32 CPUs, the rest would be wasted. As the comment indicates,
Linux supports up to 4096 CPUs, so the buffer should be much bigger, or
(ideally) the code refactored to do increm
This revision was automatically updated to reflect the committed changes.
Closed by commit R102:10d60170eaa4: Correctly show memory sizes > 4 GiB on
32 bit Linux (authored by cfeck).
REPOSITORY
R102 KInfoCenter
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D20828?vs=56992&id=57228
R
cfeck created this revision.
cfeck added a reviewer: Plasma.
cfeck added a project: Plasma.
Herald added a subscriber: plasma-devel.
cfeck requested review of this revision.
REVISION SUMMARY
BUG: 406351
REPOSITORY
R102 KInfoCenter
REVISION DETAIL
https://phabricator.kde.org/D20828
AFFECTE
This revision was automatically updated to reflect the committed changes.
Closed by commit R103:fb171dff39c7: Fix sorting to use locale-aware comparisons
(authored by jen6, committed by cfeck).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D20386?vs=55792&id=56990#toc
REPOSITORY
R103 K
https://bugs.kde.org/show_bug.cgi?id=383327
Christoph Feck changed:
What|Removed |Added
Resolution|--- |FIXED
Status|REOPENED
https://bugs.kde.org/show_bug.cgi?id=383327
--- Comment #8 from Christoph Feck ---
> I can't find the relevant bug report
Bug 70063
--
You are receiving this mail because:
You are the assignee for the bug.
cfeck added a comment.
Are the `toLower()` calls really needed? I would have expected that
`localeAwareCompare()` is case insensitive by default.
Also, please use `return x` instead of `return (x)`.
REPOSITORY
R103 KMenu Editor
REVISION DETAIL
https://phabricator.kde.org/D20386
To:
cfeck added a comment.
Hm, is moving "Minimize" one level deeper an UX improvement? Is it worth to
create a bug about this issue?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D19606
To: ngraham, #plasma, #vdg, ndavis, hein
Cc: cfeck, abetts, broulik, ndavis
cfeck requested changes to this revision.
cfeck added a comment.
This revision now requires changes to proceed.
It was suggested to change the model (C++ code? where?) instead.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D19019
To: serhiyz, #plasma, davided
cfeck added a comment.
QLineEdit has Qt::StrongFocus policy and it is the focus proxy of
KPasswordLineEdit. My impression was that KPasswordLineEdit then automatically
also had Qt::StrongFocus.
Appearantly, the policy is propagated in the other direction, so a correct
fix would be to ch
cfeck added inline comments.
INLINE COMMENTS
> Zren wrote in DigitalClock.qml:305
> This is inside a `State { when: plasmoid.formFactor ==
> PlasmaCore.Types.Vertical }`
Could you please clarify? Are changes needed?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.
cfeck added inline comments.
INLINE COMMENTS
> oxygenstyle.cpp:3273
>
> -const QColor background( palette.color( QPalette::Base ) );
> +const QBrush& background( palette.brush( QPalette::Base ) );
>
Why the reference?
REPOSITORY
R113 Oxygen Theme
REVISION DETAIL
https://bugs.kde.org/show_bug.cgi?id=348529
--- Comment #23 from Christoph Feck ---
Comment #11 works perfect for me.
--
You are receiving this mail because:
You are the assignee for the bug.
cfeck added a comment.
Could a Plasma developer please review?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D18979
To: McPain, broulik, sebas, hein
Cc: cfeck, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai,
ali-mohamed, jensreuterberg, ab
cfeck added a comment.
The status of this says 'Accepted', so developers might not check it.
If you want to wait for a developer's review, I suggest to keep the status
'Needs Review'.
REPOSITORY
R120 Plasma Workspace
BRANCH
arcpatch-D19036
REVISION DETAIL
https://phabricator.kde.
cfeck added inline comments.
INLINE COMMENTS
> breezestyle.cpp:678
> +case SH_Menu_Scrollable:
> +if (StyleConfigData::overflowingMenusMode() ==
> StyleConfigData::OM_SCROLL) {
> +return true;
if (cond) {
return true;
} else {
return
cfeck added a comment.
What he complains about is that items are unreachable. Yes, it should scroll.
Either horizontally or vertically. A scrollbar would be perfect so you see how
much you don't see. In addition to multiple columns, if there are 1½ or more.
REPOSITORY
R31 Breeze
REVISION
cfeck added a comment.
The problem is that adding this property limits to a single column. It should
show all columns, and additionally allow the user to scroll. For a menu that
only has a few more items than fit in a column this is acceptable, but for the
menu in the referenced ticket it is
cfeck added a comment.
Please not. I use wide menus extensively, e.g. for bookmarks, and are not
keen to wait eons to find an item because of the slow scrolling.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D19444
To: ngraham, #breeze, broulik
Cc: cfeck, plasma-deve
cfeck added a comment.
Disabling is preferred to hiding. If there is a widget, but not enabled, you
are eager to learn how to enable it. If you hide it, you won't even know it
exists.
REPOSITORY
R120 Plasma Workspace
BRANCH
replace-tristate-checkbox-with-combobox (branched from master)
cfeck added inline comments.
INLINE COMMENTS
> desktopicon.cpp:522
> +//don't try for too big images
> +if (img.width() > 256 || m_theme->supportsIconColoring()) {
> +return false;
Did you mean `>= 256`?
> desktopicon.cpp:545
> +if (findIt != m_monochromeHeuristics.constEnd(
cfeck added inline comments.
INLINE COMMENTS
> kmenuedit.h:60
> QSplitter *m_splitter = nullptr;
> +KTreeWidgetSearchLine* m_searchLine = nullptr;
>
Surrounding code uses `Type *var`.
> treeview.cpp:1924
> +// expand all categories if we typed more than a few characters,
> other
cfeck added a comment.
> This patch introduces support to CDB, providing useful backtraces in bug
tickets created through DrKonqi on windows.
Just noticed this. This will help a lot to get better crash reports for our
Windows ports. Thanks!
REPOSITORY
R871 DrKonqi
REVISION DETAIL
h
cfeck added a comment.
The comment is confusing. If QIcon does not resolve absolute paths, but I can
stil pass absolute paths, the comment could say that it ignores the path?
Shouldn't this comment rather be part of the README, instead of the code?
REPOSITORY
R102 KInfoCenter
REVISION DET
cfeck added a comment.
If this fixes bug 401725, please add a line "BUG: 401725" to the commit
message.
REPOSITORY
R884 Active Window Control Applet for Plasma
REVISION DETAIL
https://phabricator.kde.org/D18696
To: ntninja, Zren, majohnson
Cc: cfeck, plasma-devel, jraleigh, GB_2, ragree
cfeck added inline comments.
INLINE COMMENTS
> pulseaudio.cpp:300
> it.next();
> -if (it.value()->isVirtualDevice() || it.value()->state() !=
> state) {
> +if ((it.value()->isVirtualDevice()&& !it.value()->isDefault())
> +|| it.value()->st
cfeck added a comment.
Bug 340982 is about system-wide locale format changes. Qt currently does not
have a way to customize its internal CLDR database.
This change is indeed only about the Plasma clock, somewhat a workaround for
the bug.
REPOSITORY
R120 Plasma Workspace
REVISION DETA
cfeck added a comment.
Maybe symlinking all to the largest version?
REPOSITORY
R131 Plasma Wallpapers
BRANCH
keep-only-largest-size (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D18005
To: ngraham, #vdg, #plasma, ndavis
Cc: cfeck, ndavis, plasma-devel, kvanton, jr
On 03.01.2019 16:26, Jonathan Riddell wrote:
Today is repo freeze for the 5.15 release. I'd had no requests for
addition or removals from the repos we make tars from so I plan to use
the same list as for 5.14.
plasma-desktop-5.14.4.1.tar.xz
https://bugs.kde.org/show_bug.cgi?id=346251
Christoph Feck changed:
What|Removed |Added
Resolution|--- |WORKSFORME
Status|REPORTED
https://bugs.kde.org/show_bug.cgi?id=378823
Christoph Feck changed:
What|Removed |Added
Component|general |Picture of the Day
Product|Plasma
https://bugs.kde.org/show_bug.cgi?id=386067
--- Comment #1 from Christoph Feck ---
Are messages separated by some fixed text? Otherwise I doubt it is fixable.
--
You are receiving this mail because:
You are the assignee for the bug.
cfeck added a comment.
Or make the button have a drop-down, see e.g. "Save As..." button in
Spectacle.
REPOSITORY
R102 KInfoCenter
REVISION DETAIL
https://phabricator.kde.org/D17704
To: guoyunhe, #plasma
Cc: ngraham, gregormi, cfeck, plasma-devel, GB_2, ragreen, Pitel, ZrenBot,
lesliez
cfeck added a comment.
If I understand your patch correctly, you only forbid translating the order
of text, not the actual text labels.
REPOSITORY
R102 KInfoCenter
REVISION DETAIL
https://phabricator.kde.org/D17704
To: guoyunhe, #plasma
Cc: cfeck, plasma-devel, GB_2, ragreen, Pitel, Zre
On 27.11.2018 18:17, Jonathan Riddell wrote:
Plasma 5.14.4 is now released
https://www.kde.org/announcements/plasma-5.14.4.php
It was suggested to do a 5.14.4.1 respin of 'plasma-desktop' to include
a patch for regression bug 401508 (the bug ticket does not mention that
it can even crash the
On 19.11.2018 11:11, Roman Gilg wrote:
D1 is at coordinates (0, 0) and has logical size 3840x2160. D2 is
therefore at coordinates (3841, 0).
Is the single-pixel gap intended, or should that read (3840, 0)?
cfeck added a comment.
What is the status of this patch?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D16170
To: thsurrel, #plasma, #vdg, broulik, bruns
Cc: cfeck, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai,
ali-mohamed, jensreuterberg, ab
cfeck added a comment.
Please commit or abandon this.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D6056
To: broulik, #plasma, #vdg, davidedmundson
Cc: cfeck, anthonyfieroni, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai,
ali-mohamed, jensreuterberg,
https://bugs.kde.org/show_bug.cgi?id=399637
Bug ID: 399637
Summary: libksysguard fails to build without QtWebEngine
Product: ksysguard
Version: unspecified
Platform: Other
OS: Linux
Status: REPORTED
Severi
1 - 100 of 540 matches
Mail list logo