Feed Advanced Search

Jun 21 2020

piotrkakol added a reviewer for D29845: Added option to change grid thickness: ngraham.
Jun 21 2020, 10:48 AM · Plasma

Jun 8 2020

piotrkakol added a comment to D29845: Added option to change grid thickness.
In D29845#674882, @mart wrote:

please use gitlab on invent.kde.org for new review requests

Jun 8 2020, 3:14 PM · Plasma

Jun 4 2020

piotrkakol added a comment to D29845: Added option to change grid thickness.

@ngraham It does scale smooth-ish-ly according to taskbar size. You can see that square size (dotSize in BinaryClock.qml) is calculated based on taskbar size (height for horizontal taskbar and width for vertical taskbar). When you change taskbar size, binary clock changes size as well.

Jun 4 2020, 2:18 AM · Plasma
piotrkakol added a comment to D29845: Added option to change grid thickness.

This is probably the last diff from my initial Binary Clock rework from D13937.
If you have any ideas what else can be improved after these changes, please let me know.

Jun 4 2020, 12:03 AM · Plasma
piotrkakol updated the summary of D29845: Added option to change grid thickness.
Jun 4 2020, 12:00 AM · Plasma

Jun 3 2020

piotrkakol requested review of D29845: Added option to change grid thickness.
Jun 3 2020, 11:57 PM · Plasma

May 24 2020

piotrkakol abandoned D14384: Renamed dot to square in variables and filenames.
May 24 2020, 9:24 PM · Plasma

Dec 18 2019

piotrkakol added a comment to D25642: Add minimal number of moves statistics.

Yes, I have tried it. At first I just launched the game and checked if the functionality worked. After your last comment I deleted all files and checked all scenarios I could think of from the start. I'm sorry that I didn't do it before I submitted this diff. I didn't think about the initial state of the game.

Dec 18 2019, 9:27 PM

Dec 3 2019

piotrkakol updated the diff for D25642: Add minimal number of moves statistics.
Dec 3 2019, 2:26 PM

Dec 2 2019

piotrkakol updated the diff for D25642: Add minimal number of moves statistics.

Changed the default value set to minmoves from 0 to -1 thanks to aacid's comment.

Dec 2 2019, 12:26 AM

Dec 1 2019

piotrkakol updated the summary of D25642: Add minimal number of moves statistics.
Dec 1 2019, 1:15 AM
piotrkakol requested review of D25642: Add minimal number of moves statistics.
Dec 1 2019, 1:10 AM

Oct 1 2018

piotrkakol added a comment to D15876: Fix copyrights in binary clock files.

Well, previously it looked like I claimed to be contributor to fuzzy-clock, analog-clock and version from KDE4.
Now it's clear that I only contributed to the newest version of binary-clock.

Oct 1 2018, 1:01 PM · Plasma
piotrkakol requested review of D15876: Fix copyrights in binary clock files.
Oct 1 2018, 12:12 PM · Plasma

Sep 5 2018

piotrkakol added a comment to D14384: Renamed dot to square in variables and filenames.

Yes, but I forgeted to put them in the last diff. Do I really have to make a separate diff to change it?

Sep 5 2018, 1:58 PM · Plasma
piotrkakol added a comment to D14384: Renamed dot to square in variables and filenames.

I did change main.qml as you can see here:
https://phabricator.kde.org/R114:9eae090a3038ebf97ffeee8c9b773a8610e13021

Sep 5 2018, 1:49 PM · Plasma
piotrkakol added a comment to D14384: Renamed dot to square in variables and filenames.

Ping.

Sep 5 2018, 1:09 PM · Plasma

Jul 25 2018

piotrkakol requested review of D14384: Renamed dot to square in variables and filenames.
Jul 25 2018, 8:53 PM · Plasma
piotrkakol requested review of D14377: Changed variable type to int to display squares instead of rectangles.
Jul 25 2018, 3:58 PM · Plasma

Jul 23 2018

piotrkakol requested review of D14307: Changed refresh time from 30 to 60 seconds.
Jul 23 2018, 6:32 PM · Plasma
piotrkakol abandoned D14306: Changed refresh time from 30 to 60 seconds.
Jul 23 2018, 5:55 PM · Plasma
piotrkakol requested review of D14306: Changed refresh time from 30 to 60 seconds.
Jul 23 2018, 5:53 PM · Plasma

Jul 21 2018

piotrkakol abandoned D14268: Changed refresh time from 30 to 60 seconds.
Jul 21 2018, 10:38 PM · Plasma
piotrkakol added a comment to D14268: Changed refresh time from 30 to 60 seconds.

Thanks, David.
Yeah, even though I followed the instructions you can see that the branch in this diff is "change-refresh-time (branched from master)" and it should be "change-refresh-time (branched from code-cleanup)".
I'll remove this diff and try to make another one from the correct branch on Monday, with the line you requested.

Jul 21 2018, 10:24 PM · Plasma
piotrkakol requested review of D14268: Changed refresh time from 30 to 60 seconds.
Jul 21 2018, 10:06 PM · Plasma
piotrkakol requested review of D14267: Cleaned code for binary clock applet and added comments.
Jul 21 2018, 9:06 PM · Plasma

Jul 20 2018

piotrkakol added a comment to D13937: Update formulas and add option to change grid thickness.

Thanks for the link! I'll check it out probably tomorrow and will make new diff(s). :-)

Jul 20 2018, 3:25 PM · Plasma
piotrkakol added a comment to D13937: Update formulas and add option to change grid thickness.

Thanks. It can be breaked into smaller pieces. In fact, these 6 lines in the summary are titles of my commits that artisan combined into 1 diff.
This is why Phabricator is quite inconvenient. On GitHub you can have multiple commits in one Pull Request (diff equivalent), committer doesn't need to have merge rights, so a repo maintainer who has merges the PR.
E.g. here there are 2 commits in 1 PR.
Here I can't make commits - only diffs. And from what I understand 1 diff should be 1 commit. So do I really need to make 1 diff for the 1st commit, then wait for it to be merged, add 2nd diff, etc. up to 6th diff/commit? I could make 6 diffs now but some of them would have to include changes from other diffs. And I don't know if diffs can have parents.

Jul 20 2018, 3:09 PM · Plasma

Jul 7 2018

piotrkakol added a comment to D13937: Update formulas and add option to change grid thickness.

I don't have KDE developer account so I cannot push this patch.

Jul 7 2018, 11:20 PM · Plasma
piotrkakol added a comment to D13937: Update formulas and add option to change grid thickness.


On the left there's the old applet and on the right mine after only changing dotSize (now squareSize) from real to int. All squares, not rectangles.

Jul 7 2018, 10:45 PM · Plasma
piotrkakol requested review of D13937: Update formulas and add option to change grid thickness.
Jul 7 2018, 10:27 PM · Plasma

Mar 7 2018

piotrkakol added a comment to D10434: Allow users to change the format to BCD (decimal) in Binary Clock widget.

I was instructed to give my contact details that will be included in the commit message.
Piotr Kąkol piotrkakol@pm.me

Mar 7 2018, 2:46 PM · Plasma

Feb 19 2018

piotrkakol added a comment to D10434: Allow users to change the format to BCD (decimal) in Binary Clock widget.

Thank you for reviewing.

Feb 19 2018, 4:54 PM · Plasma

Feb 12 2018

piotrkakol updated the diff for D10434: Allow users to change the format to BCD (decimal) in Binary Clock widget.

Instead of changing the base to decimal I decided to allow users to choose between base 10 (BCD format) and 16 (sexagesimal format) in the config.

Feb 12 2018, 8:19 PM · Plasma

Feb 11 2018

piotrkakol updated the summary of D10434: Allow users to change the format to BCD (decimal) in Binary Clock widget.
Feb 11 2018, 12:10 AM · Plasma
piotrkakol updated the summary of D10434: Allow users to change the format to BCD (decimal) in Binary Clock widget.
Feb 11 2018, 12:05 AM · Plasma
piotrkakol requested review of D10434: Allow users to change the format to BCD (decimal) in Binary Clock widget.
Feb 11 2018, 12:02 AM · Plasma