xyquadrat (Julian Schraner)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Jan 3 2017, 11:01 AM (75 w, 6 d)
Availability
Available

Recent Activity

Sat, Jun 2

xyquadrat updated the task description for T8552: Polish Open/Save dialogs.
Sat, Jun 2, 1:34 PM · VDG

Wed, May 30

xyquadrat added a reviewer for D13213: Add possibility to sort by descending order: Gwenview.

I think you forgot to specify reviewers, so I added them for you.

Wed, May 30, 2:34 PM · Gwenview

Mon, May 21

xyquadrat added a comment to T6832: Improvements to Bugzilla - Making it easier and simpler.

Thanks @lueck for pointing these applications out; I have now re-added them to my active list.

Mon, May 21, 7:33 PM · Goal settings 2017

Sun, May 20

xyquadrat added a comment to T6832: Improvements to Bugzilla - Making it easier and simpler.

Hm, is there a compelling reason to remove obsolete products from the database? If they are marked as obsolete, no new bugs can be filed against it anyway, and those should not show up in a browsable list for the user. Of course if you have admin rights they will show, but how often does an admin have to browse a product list without a filter?

I must admit that I just realized that the list of apps you can report a bug against is not the same as the ones accessible through clicking "Browse"...
But I think we should consider hiding the obsolete products not only when reporting bugs, but also in a) the advanced search & b) in the list from Browse. It'd simplify the search process for triagers a lot. Additionally, there are some products which are obsolete but you can still file bugs against them (e.g kaveau).

Sun, May 20, 2:42 PM · Goal settings 2017
xyquadrat added a reviewer for D12983: (icon view) Text width relative to icon size: Dolphin.

You forgot to specify any reviewers, so I added them for you. +1 from my side, removing code while improving the user experience is always a good thing :)

Sun, May 20, 12:56 PM · Dolphin

May 19 2018

xyquadrat added a comment to T6832: Improvements to Bugzilla - Making it easier and simpler.

I decided to do something remotely useful with my free time and sifted through all applications/libraries present on bugs.kde.org, removing most obsolete products. My guidelines to determine whether a product was deprecated were:

  1. Is the product not marked as "unmaintained"/"deprecated"/"replaced"?
  2. Is the product in the repositories of Ubuntu LTS?
  3. Is the product on cgit.kde.org?
  4. If it is on cgit.kde.org, does it have any activity in the last year?
May 19 2018, 5:17 PM · Goal settings 2017

Apr 28 2018

xyquadrat added a comment to D11022: Resize the "normal" folder icons.

Yeah, pretty much ;) Nah, real life has been busy and I think that it'll take at least another two weeks until I can pick this up again. Frankly, I don't even really know what to implement now... Assuming that we do not want to WONTFIX the bug, we need to shrink the preview size down. @sharvey suggested that we set this to about 85% of the panel size, but this would not resolve the enormous icons if the panel is very wide. Should we introduce a cap (such as two times Size::Enormous)?

Apr 28 2018, 12:17 PM

Apr 27 2018

xyquadrat added a comment to D12558: Don't filter out duplicated entries from places panel.

Well, do we still need this code now? Are there any users which have not completed the transition yet? If yes, then we should try to find a way to still avoid these duplicates while preventing this crash.

Apr 27 2018, 5:29 PM

Apr 22 2018

xyquadrat added a comment to T8554: Compile list of questions for marketing research company.
In T8554#139259, @paulb wrote:
  • Have you heard of KDE or specific KDE applications before?

Do you think asking about something specific, depending on context, may work better?:

Apr 22 2018, 3:10 PM · KDE Promo

Apr 19 2018

xyquadrat added a comment to D12321: Hide file preview when icon is too small.

Fabulous. I agree with @rkflx that before committing this, we should also prepare a similar patch for Dolphin to unify the behaviors.

Apr 19 2018, 7:52 PM · Frameworks
xyquadrat added a comment to T8554: Compile list of questions for marketing research company.

+1 for asking institutions (especially in education). KDE has some great educational apps (GCompris, KStars, KGeography...) and there are even special distributions designed for schools (e.g. the "lernstick", a german project that is pretty much Debian with tons of edu-apps added in).

Apr 19 2018, 6:47 PM · KDE Promo
xyquadrat added a comment to T8428: Update release notes for Dolphin 18.04 release.

@ngraham Quick reminder: You can now mark this task as resolved as the 18.04 release happened today. Wohoo! ;)

Apr 19 2018, 4:22 PM · Dolphin

Apr 11 2018

xyquadrat added a comment to T8428: Update release notes for Dolphin 18.04 release.

We can take some screenshots from your blog posts, but the question is how many screenshots are necessary/useful. I think that having three might be a good number, one for the "Empty Trash" button, one for the dockable Terminal plus either sorting by "Date Photographed" or the git integration. Thoughts?

Apr 11 2018, 7:50 PM · Dolphin
xyquadrat accepted D12119: Fix untranslated places panel entries.

Even though I wasn't able to reproduce the bug itself (probably because of some leftover old files), this patch is really needed as it'd be very bad if suddenly all places entries were displayed untranslated in 18.04. The patch itself seems reasonable (not a lot of changes -> not a lot to complain about), so +1 from me.

Apr 11 2018, 6:07 PM

Apr 10 2018

xyquadrat added a comment to T6832: Improvements to Bugzilla - Making it easier and simpler.

And more about bugzilla's usefulness here:
https://mail.kde.org/pipermail/kde-community/2018q1/004274.html

Apr 10 2018, 4:58 PM · Goal settings 2017
xyquadrat added a comment to T8473: Dolphin 'Find' button behavior.

Okay, so the functionality we are discussing is already implemented with D10246, but this patch suffers from the same problem. The patch does not apply cleanly for me, so I wasn't able to fiddle around with it. But theoretically, if we could resolve the remaining problem (Ctrl + F also closes the search box), we could simply use D10246 and the wish from bug 353227 would be fulfilled.

Apr 10 2018, 8:50 AM · Dolphin

Apr 9 2018

xyquadrat added a comment to T8473: Dolphin 'Find' button behavior.

Additionally, opening up a new tab could also cause more problems...
We would need to come up with a reliable way of detecting whether the search box is opened in a specific panel or not & update the state of the button respectively. I think we should still strive to implement this (even if this would be more complicated), as the feature itself is rather useful for the UX & we could also consider removing the [X] that currently looks a bit off.

Apr 9 2018, 8:07 PM · Dolphin

Apr 3 2018

xyquadrat added a comment to T7894: Dolphin uses the F11 key for something that's not "Enter full screen".

Apparently I misunderstood the goal of the task... Anyways, I guess I'll just wait until you made a decision (if the final result would be to change it, I'd do this if someone wants me to).

Apr 3 2018, 5:32 PM · Dolphin
xyquadrat added a comment to T7894: Dolphin uses the F11 key for something that's not "Enter full screen".

Hmm, I cannot really see the "other way" you are proposing... Do you want to simply change the F11 key, or not change anything at all (as we do not have a Full Screen feature right now)?

Apr 3 2018, 2:42 PM · Dolphin

Apr 2 2018

xyquadrat added a comment to D11022: Resize the "normal" folder icons.

I agree that the large icon might look too big for wide panels, but I guess the only really good way to optimize for such a corner case would be to increase the font size as well (which is another task, and might look bad).

Apr 2 2018, 2:56 PM
xyquadrat added a comment to T7894: Dolphin uses the F11 key for something that's not "Enter full screen".

Thinking about this for a bit, I tried to come up with a function keys setup which should be more consistent/make more sense (changed ones are marked in italics):

Apr 2 2018, 1:16 PM · Dolphin

Apr 1 2018

xyquadrat added a comment to D11022: Resize the "normal" folder icons.

Still, as I mentioned in bugzilla, the only inconsistency that I'd fix is that the information panel does not check whether the preview setting is enabled.

Apr 1 2018, 10:17 AM

Mar 31 2018

xyquadrat added inline comments to D11816: Split Image Size into Width/Height.
Mar 31 2018, 7:42 PM
xyquadrat updated the diff for D11816: Split Image Size into Width/Height.

Clean up references to imageSize

Mar 31 2018, 1:05 PM
xyquadrat updated the diff for D11816: Split Image Size into Width/Height.

Merge the ratingRole with widthRole/heightRole

Mar 31 2018, 10:53 AM

Mar 30 2018

xyquadrat requested review of D11816: Split Image Size into Width/Height.
Mar 30 2018, 7:17 PM
xyquadrat updated the diff for D11022: Resize the "normal" folder icons.

I am apparently not able to control arcanist... reverting changes which should have been in a different patch.

Mar 30 2018, 7:04 PM
xyquadrat updated the diff for D11022: Resize the "normal" folder icons.
Mar 30 2018, 6:56 PM

Mar 27 2018

xyquadrat added a comment to D11022: Resize the "normal" folder icons.

Well, then we can simply keep the whole animation (as the animation only triggers if the size of the pixmap changes), adjusting it to only work if the panel is resized would probably require more code than the current solution.

Mar 27 2018, 5:47 PM
xyquadrat added a comment to D11022: Resize the "normal" folder icons.

If we make all icons of the same size (which I'm not sure I'd like), that code would become useless and should then be removed.

Mar 27 2018, 4:21 PM
xyquadrat updated the diff for D11022: Resize the "normal" folder icons.

Also adapt the size of all "Places" icons

Mar 27 2018, 4:18 PM

Mar 25 2018

xyquadrat added a comment to D11022: Resize the "normal" folder icons.

The hardcoded value is now removed.

Mar 25 2018, 7:32 PM
xyquadrat updated the diff for D11022: Resize the "normal" folder icons.

Remove hardcoded value

Mar 25 2018, 7:31 PM
xyquadrat abandoned D11367: Capitalize first letter of type.

Should be fixed on a higher level (translations) -> abandoning patch.

Mar 25 2018, 7:07 PM · Baloo

Mar 16 2018

xyquadrat added a comment to D11367: Capitalize first letter of type.

This appears to be more complicated than I thought, thanks for the hint @fabiank . I think the best solution would be to notify the translators somehow, so that they can update the respective strings.
But I am not familiar with the translation process of KDE, so I don't know how to contact them... I guess I'll abandon this patch soon because it tries to fix a problem which should be resolved on a higher level.

Mar 16 2018, 8:17 PM · Baloo
xyquadrat added a comment to D11367: Capitalize first letter of type.

US English, I guess the German translator already capitalized "Ordner" as this is the correct way in German.

Mar 16 2018, 6:16 PM · Baloo

Mar 15 2018

xyquadrat updated the summary of D11367: Capitalize first letter of type.
Mar 15 2018, 8:06 PM · Baloo
xyquadrat requested review of D11367: Capitalize first letter of type.
Mar 15 2018, 8:04 PM · Baloo

Mar 6 2018

xyquadrat added inline comments to D11022: Resize the "normal" folder icons.
Mar 6 2018, 5:30 PM
xyquadrat added a comment to D11022: Resize the "normal" folder icons.

I must say I don't really like the larger icon. How does that change the appearance of all the other file icons in the panel?

Mar 6 2018, 5:27 PM

Mar 4 2018

xyquadrat abandoned D10609: Make "Split" checkable.

Abandoning because the behavior before the patch was the actually intended one.

Mar 4 2018, 4:15 PM
xyquadrat updated the summary of D11022: Resize the "normal" folder icons.
Mar 4 2018, 4:06 PM
xyquadrat updated the summary of D11022: Resize the "normal" folder icons.
Mar 4 2018, 4:02 PM
xyquadrat requested review of D11022: Resize the "normal" folder icons.
Mar 4 2018, 3:57 PM

Feb 17 2018

xyquadrat added a comment to D10609: Make "Split" checkable.

The way I am currently attempting to fix this bug is very inconsistent... I realized that it does not play well with the startup split view option.

Feb 17 2018, 4:52 PM
xyquadrat updated the summary of D10609: Make "Split" checkable.
Feb 17 2018, 4:44 PM
xyquadrat updated the summary of D10609: Make "Split" checkable.
Feb 17 2018, 4:21 PM
xyquadrat requested review of D10609: Make "Split" checkable.
Feb 17 2018, 4:18 PM

Jan 23 2018

xyquadrat added a comment to D10059: Add option to sort for original date.

Please use FEATURE: rather than BUG: in the commit message.

Jan 23 2018, 8:12 PM
xyquadrat updated the summary of D10059: Add option to sort for original date.
Jan 23 2018, 8:11 PM
xyquadrat added a comment to D10059: Add option to sort for original date.

Looks great. Any chance you can attach a screenshot to the Summary section, too?

Jan 23 2018, 7:50 PM
xyquadrat updated the summary of D10059: Add option to sort for original date.
Jan 23 2018, 7:50 PM
xyquadrat updated the summary of D10059: Add option to sort for original date.
Jan 23 2018, 7:48 PM
xyquadrat updated the diff for D10059: Add option to sort for original date.

Change name from "Original Date" to "Date Photographed"

Jan 23 2018, 7:16 PM
xyquadrat updated the summary of D10059: Add option to sort for original date.
Jan 23 2018, 7:06 PM

Dec 2 2017

xyquadrat updated the diff for D8273: Change window title when searching.

Forgot to commit changes before generating the diff...

Dec 2 2017, 8:27 AM · Dolphin

Dec 1 2017

xyquadrat updated the diff for D8273: Change window title when searching.

Removed emptySearch variable, moved "Empty Search" to inline function.

Dec 1 2017, 3:27 PM · Dolphin

Nov 23 2017

xyquadrat updated the diff for D8273: Change window title when searching.

Window title is now "Empty search" when the user hasn't input any search term.

Nov 23 2017, 5:55 PM · Dolphin

Nov 13 2017

xyquadrat added inline comments to D8273: Change window title when searching.
Nov 13 2017, 7:02 AM · Dolphin

Nov 12 2017

xyquadrat added inline comments to D8273: Change window title when searching.
Nov 12 2017, 7:15 PM · Dolphin
xyquadrat updated the diff for D8273: Change window title when searching.

Removed the facetType() and currentFacet() function as proposed by @elvisangelaccio. Merged the search and text variable into searchText, which now uses %1 to add in the current search text for better localization. But I am not entirely sure if the consent is that we remove the facets from the title (which gives more space) or if I should keep it (which gives more information to the user) but adapt to the other comments.

Nov 12 2017, 7:14 PM · Dolphin
xyquadrat updated the diff for D8273: Change window title when searching.

@elvisangelaccio The basic patch is the same, but the "Places" feature has been added & some code style changes were made.

Nov 12 2017, 4:41 PM · Dolphin
xyquadrat added a comment to D8273: Change window title when searching.

Hmm, I tested it multiple times (also with a completely fresh copy & applying the patch) and it always worked for me.

Nov 12 2017, 2:44 PM · Dolphin
xyquadrat updated the diff for D8273: Change window title when searching.

I am not very experienced with git and arc , but I think that everything should work now. Sorry for all the trouble with this patch.

Nov 12 2017, 11:50 AM · Dolphin

Nov 11 2017

xyquadrat added a comment to D8273: Change window title when searching.

Weird, when I do

arc patch D8273

on the lastest master, the patch applies perfectly fine for me.

Nov 11 2017, 3:09 PM · Dolphin

Nov 10 2017

xyquadrat updated the diff for D8273: Change window title when searching.

This revision adds the feature proposed by @ngraham. The search text will now change to "Search for [type] named [input]" or "Search for [type]" if the user did not input a search term.

Nov 10 2017, 8:42 PM · Dolphin

Nov 7 2017

xyquadrat updated the diff for D8273: Change window title when searching.
  • Changed "Searching [user input]" to "Search for [user input]"
  • Instead of using isSearchModeEnabled() I now use viewPropertyContext(), although I am not 100% sure if that will work with all corner cases (@emmanuelp probably knows that better)
Nov 7 2017, 8:22 PM · Dolphin

Oct 28 2017

xyquadrat added a comment to D8273: Change window title when searching.

@xyquadrat, have you had a chance to look into any of the comments folks have posted?

Oct 28 2017, 3:56 PM · Dolphin

Oct 15 2017

xyquadrat added a comment to D8273: Change window title when searching.

@emmanuelp Am I correct if I assume that I can then delete the changes made in dolphinviewcontainer.cpp/.h as the currentSearchText() function would then no longer be needed?

Oct 15 2017, 5:56 PM · Dolphin

Oct 14 2017

xyquadrat updated the diff for D8273: Change window title when searching.

Split the "caption" variable into two variables (schemePrefix & fileName) to reflect changes made from master since I sent this to review for the first time.

Oct 14 2017, 9:09 AM · Dolphin

Oct 13 2017

xyquadrat created D8273: Change window title when searching.
Oct 13 2017, 6:20 AM · Dolphin