ngraham (Nathaniel Graham)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Apr 15 2017, 7:18 PM (22 w, 6 d)
Availability
Available

Recent Activity

Today

ngraham added reviewers for D7948: Only match real MIME types, not e.g. "raw CD image": Frameworks, nicolasfella, dfaure, kossebau, vhanda.
Sat, Sep 23, 4:40 AM · Frameworks
ngraham created D7948: Only match real MIME types, not e.g. "raw CD image".
Sat, Sep 23, 4:38 AM · Frameworks
ngraham added a comment to D7446: Add a Recent Documents places item to Dolphin and file pickers by default.

It seems odd to have all of these special KIO URLs that we don't actually want to use because they're rough and underdeveloped. They're rough and underdeveloped because they're hidden by default, so nobody sees them, and nobody files bugs or submits patches for them. But I do see your point.

Sat, Sep 23, 2:33 AM · Frameworks, Dolphin
ngraham added a comment to D7944: Pre-select navigation bar URL when clicking on it to enter edit mode.

Opps, I've corrected the bug number.

Sat, Sep 23, 1:39 AM · Frameworks
ngraham updated the summary of D7944: Pre-select navigation bar URL when clicking on it to enter edit mode.
Sat, Sep 23, 12:49 AM · Frameworks

Yesterday

ngraham added reviewers for D7944: Pre-select navigation bar URL when clicking on it to enter edit mode: KDE Applications, Frameworks, broulik, dfaure.
Fri, Sep 22, 9:23 PM · Frameworks
ngraham created D7944: Pre-select navigation bar URL when clicking on it to enter edit mode.
Fri, Sep 22, 9:21 PM · Frameworks
ngraham added a comment to D7446: Add a Recent Documents places item to Dolphin and file pickers by default.

I don't think this is "semi-useful". " A Recent Documents feature in the file manager and open/save dialogs is IMHO really important, especially for lesst-technical users who use features like this on other platforms expensively instead of making extensive use of folder hierarchies. "The user can add it" is a problematic response since 99.9% of users don't know this exists, and therefore don't know that this functionality is available in the first place (and if they did, they would find it challenging to add).

Fri, Sep 22, 9:00 PM · Frameworks, Dolphin
ngraham requested changes to D7647: Bug 205157 - two clicks on file to rename.

Actually, could you edit the Summary as follows?

Fri, Sep 22, 6:35 PM
ngraham accepted D7647: Bug 205157 - two clicks on file to rename.

This works really well in my testing. I like it. Let's implement rename-with-long-press in another patch.

Fri, Sep 22, 6:33 PM
ngraham added a comment to D7446: Add a Recent Documents places item to Dolphin and file pickers by default.

Anybody object?

Fri, Sep 22, 6:18 PM · Frameworks, Dolphin
ngraham committed R260:144f440cb76c: Always show image rotation buttons in Image Operations sidebar when other… (authored by ngraham).
Always show image rotation buttons in Image Operations sidebar when other…
Fri, Sep 22, 4:03 PM
ngraham closed D7743: Always show image rotation buttons in Image Operations sidebar when other operations are available.
Fri, Sep 22, 3:59 PM
ngraham updated the diff for D7743: Always show image rotation buttons in Image Operations sidebar when other operations are available.

Removing these lines entirely instead of commenting them out

Fri, Sep 22, 3:58 PM
ngraham added a comment to D7743: Always show image rotation buttons in Image Operations sidebar when other operations are available.

Any more opinions?

Fri, Sep 22, 3:44 PM
ngraham committed R223:15917a42608c: Merge remote-tracking branch 'origin/Applications/17.08' (authored by ngraham).
Merge remote-tracking branch 'origin/Applications/17.08'
Fri, Sep 22, 3:41 PM
ngraham committed R223:e80972e18909: Fix sidebar labels being unreadable when selected or hovered over (authored by ngraham).
Fix sidebar labels being unreadable when selected or hovered over
Fri, Sep 22, 3:40 PM
ngraham closed D7793: Fix sidebar labels being unreadable when selected or hovered over.
Fri, Sep 22, 3:39 PM · Okular
ngraham updated the task description for T7050: Privacy Software.
Fri, Sep 22, 1:59 PM · Goal settings 2017
ngraham added a comment to D7793: Fix sidebar labels being unreadable when selected or hovered over.

I managed to get arc set up. Am I basically going to do this?

Fri, Sep 22, 5:12 AM · Okular
ngraham committed R318:76698ff82cf5: Add Bitrate to Dolphin's Additional Information (authored by ngraham).
Add Bitrate to Dolphin's Additional Information
Fri, Sep 22, 4:56 AM
ngraham closed D7763: Add Bitrate to Dolphin's Additional Information.
Fri, Sep 22, 4:56 AM · Dolphin
ngraham updated the diff for D7763: Add Bitrate to Dolphin's Additional Information.

Updating diff to try to get it to apply with arc (will use arc for future revisions)

Fri, Sep 22, 4:53 AM · Dolphin
ngraham committed R134:4d9461f73b6c: "Close Description" tooltip should instead say "Back" (authored by ngraham).
"Close Description" tooltip should instead say "Back"
Fri, Sep 22, 4:33 AM
ngraham committed R134:98c1d5ad33d6: Remove launch feature from hamburger button and restore to the toolbar (authored by ngraham).
Remove launch feature from hamburger button and restore to the toolbar
Fri, Sep 22, 4:33 AM
ngraham closed D7904: "Close Description" tooltip should instead say "Back".
Fri, Sep 22, 4:33 AM · Plasma, Discover Software Store
ngraham closed D7905: Remove launch feature from hamburger button and restore to the toolbar.
Fri, Sep 22, 4:31 AM · Discover Software Store, Plasma
ngraham added a comment to D7929: [WIP] Add new Column View option to KDirOperator.

I'll just leave this here: ;-) https://bugs.kde.org/show_bug.cgi?id=290747

Fri, Sep 22, 4:09 AM · Frameworks

Thu, Sep 21

ngraham added a comment to D7914: Select and scroll to previous folder when going Up.

Yeah. I don't have a Reviewboard account, and it seems a bit silly to create one seeing as how it's going away soon. Anyone wanna nuke it?

Thu, Sep 21, 9:36 PM · Dolphin
ngraham added a comment to D7905: Remove launch feature from hamburger button and restore to the toolbar.

Excellent. Let's continue that discussion in a bug report I just filed to avoid losing it once I can commit the patch (hopefully later tonight):

Thu, Sep 21, 3:51 PM · Discover Software Store, Plasma
ngraham added a comment to D7905: Remove launch feature from hamburger button and restore to the toolbar.

Yes, I think that makes sense, but it wouldn't be quite enough, since Plasma ships with three UI options for the launcher button, and Application Dashboard is only one of them. Each one I think needs to prominently highlight recently-installed applications.

Thu, Sep 21, 3:45 PM · Discover Software Store, Plasma
ngraham added a comment to D7700: Show list of tags in PlacesView.

@emmanuelp Ping! It would be great to get your blessing so we can land this.

Thu, Sep 21, 3:34 PM · Dolphin
ngraham added a reviewer for D7743: Always show image rotation buttons in Image Operations sidebar when other operations are available: jensreuterberg.
Thu, Sep 21, 3:33 PM
ngraham added a comment to D7905: Remove launch feature from hamburger button and restore to the toolbar.

I quite agree that we should make launching programs more discoverable in other ways--especially launching new programs ("I just installed this thingy; now where is it!?"). I've recently been thinking about that myself, in fact. In Ubuntu Unity, newly-installed programs get added right to the Dock. In Windows, they appears at the top of the Start Menu. GNOME and macOS have sub-optimal UX here.

Thu, Sep 21, 3:30 PM · Discover Software Store, Plasma
ngraham abandoned D7914: Select and scroll to previous folder when going Up.

Fair enough. Thanks for merging the other patch! Abandoning this one.

Thu, Sep 21, 3:05 PM · Dolphin
ngraham added a comment to D7905: Remove launch feature from hamburger button and restore to the toolbar.

I strongly believe that having a visible launch button is critical. The launch button may be meaningless fluff for me, and you, and other developers, but developers don't need to use GUI software center apps. We're not the intended audience.

Thu, Sep 21, 2:56 PM · Discover Software Store, Plasma
ngraham updated the summary of D7914: Select and scroll to previous folder when going Up.
Thu, Sep 21, 1:17 PM · Dolphin
ngraham created D7914: Select and scroll to previous folder when going Up.
Thu, Sep 21, 1:16 PM · Dolphin
ngraham added a comment to D7793: Fix sidebar labels being unreadable when selected or hovered over.

Thanks Henrik! Yes, I have a developer account and should be able to commit now. I haven't had the time to actually set that up yet though. I'll look into it late tonight or tomorrow.

Thu, Sep 21, 12:57 PM · Okular
ngraham retitled D7793: Fix sidebar labels being unreadable when selected or hovered over from Improve text visibility for Okular sidebar tabs that are hovered-over or selected-but-out-of-focus when using Breeze color scheme to Fix sidebar labels being unreadable when selected or hovered over.
Thu, Sep 21, 12:55 PM · Okular
ngraham abandoned D7906: Locate the Back button correctly in the toolbar.

I suspected as much. Abandoning.

Thu, Sep 21, 12:51 PM · Discover Software Store, Plasma
ngraham created D7906: Locate the Back button correctly in the toolbar.
Thu, Sep 21, 5:25 AM · Discover Software Store, Plasma
ngraham retitled D7905: Remove launch feature from hamburger button and restore to the toolbar from Remove launch feature from hamburger button to Remove launch feature from hamburger button and restore to the toolbar.
Thu, Sep 21, 5:04 AM · Discover Software Store, Plasma
ngraham created D7905: Remove launch feature from hamburger button and restore to the toolbar.
Thu, Sep 21, 5:03 AM · Discover Software Store, Plasma
ngraham created D7904: "Close Description" tooltip should instead say "Back".
Thu, Sep 21, 4:46 AM · Plasma, Discover Software Store
ngraham updated the diff for D7793: Fix sidebar labels being unreadable when selected or hovered over.

Missed one more tiny formatting issue

Thu, Sep 21, 4:25 AM · Okular
ngraham updated the diff for D7793: Fix sidebar labels being unreadable when selected or hovered over.

Removed unnecessary parentheses and cleaned up the summary/commit message

Thu, Sep 21, 4:23 AM · Okular

Wed, Sep 20

ngraham added reviewers for D7810: Add missing icon sizes: VDG, jensreuterberg.
Wed, Sep 20, 2:23 AM · Breeze, Frameworks

Tue, Sep 19

ngraham updated the diff for D7763: Add Bitrate to Dolphin's Additional Information.

@aacid, does it apply now for you?

Tue, Sep 19, 4:29 AM · Dolphin
ngraham added a comment to D7793: Fix sidebar labels being unreadable when selected or hovered over.

Oh I see. Here's how it looks with the Oxygen style:

Tue, Sep 19, 4:06 AM · Okular

Mon, Sep 18

ngraham added a comment to D7805: Add a splash screen to Marble Maps.

I approve, FWIW.

Mon, Sep 18, 5:40 PM · Marble
ngraham added a comment to D7805: Add a splash screen to Marble Maps.

Lovely, much better.

Mon, Sep 18, 5:32 PM · Marble

Sun, Sep 17

ngraham added a comment to D7700: Show list of tags in PlacesView.

Looks good to me. @anthonyfieroni?

Sun, Sep 17, 11:14 PM · Dolphin

Sat, Sep 16

ngraham added a comment to D7714: Show dialog to ask when closing when more than tab open.

So for the record, I agree with folks that our "are you sure you want to close all these tabs" dialogs could stand for a bit of usability polishing.

Sat, Sep 16, 1:34 PM · Okular

Fri, Sep 15

ngraham added a comment to D7805: Add a splash screen to Marble Maps.

Does the SVG version look any better?

Fri, Sep 15, 9:13 PM · Marble
ngraham added a comment to T6832: Improve bugzilla process for new users.

Sounds good to me!

Fri, Sep 15, 7:36 PM · Goal settings 2017
ngraham added a comment to D7805: Add a splash screen to Marble Maps.

@sanjibanb, can you update the new splash screen to use the SVG that Torsten is referring to?

Fri, Sep 15, 4:25 PM · Marble
ngraham accepted D7805: Add a splash screen to Marble Maps.

Great!

Fri, Sep 15, 4:15 PM · Marble
ngraham added a comment to D7805: Add a splash screen to Marble Maps.

I rather prefer 100x100 since it looks nice and sharp, but would accept 150x150 if other folks think it's okay.

Fri, Sep 15, 3:56 PM · Marble
ngraham added a comment to D7805: Add a splash screen to Marble Maps.

That still looks blurry to me, though it's not your fault. The source 192x192 image itself looks blurry, as if it's been scaled up. Shame we don't have a better icon, or it's not an SVG. If you can change the size of the icon to the point where it doesn't look blurry, I'll approve it.

Fri, Sep 15, 3:34 PM · Marble
ngraham requested changes to D7805: Add a splash screen to Marble Maps.

The size is much better, but now it's rather blurry. Is there a high-res icon we can use instead?

Fri, Sep 15, 2:04 PM · Marble
ngraham accepted D6312: Keep renamed file(s) in view.

Looks good to me too.

Fri, Sep 15, 2:58 AM · Dolphin
ngraham added a comment to D7714: Show dialog to ask when closing when more than tab open.

I'm willing to commandeer this patch and with with folks to do the wording change. Do you approve, Albert?

Fri, Sep 15, 2:53 AM · Okular
ngraham added reviewers for D7714: Show dialog to ask when closing when more than tab open: Okular, KDE Applications.
Fri, Sep 15, 2:51 AM · Okular
ngraham added a comment to D7793: Fix sidebar labels being unreadable when selected or hovered over.

I did test with Oxygen, as that's a built-in theme. It looks fine:

Fri, Sep 15, 2:47 AM · Okular

Thu, Sep 14

ngraham added a comment to D7805: Add a splash screen to Marble Maps.

Can you add a screenshot of what this looks like, maybe?

Thu, Sep 14, 3:23 AM · Marble

Wed, Sep 13

ngraham accepted D7130: KMoreTools: Update and fix desktopfiles.

FWIW, this looks good to me.

Wed, Sep 13, 11:52 PM · Frameworks
ngraham added a comment to D7776: Add "Release Year" column to Dolphin's Details view.

@alexeymin No, I don't have commit access. I have applied for a developer account, but am still waiting to hear back. In the meantime, would you mind committing it for me?

Wed, Sep 13, 4:44 AM
ngraham updated the test plan for D7793: Fix sidebar labels being unreadable when selected or hovered over.
Wed, Sep 13, 4:42 AM · Okular
ngraham created D7793: Fix sidebar labels being unreadable when selected or hovered over.
Wed, Sep 13, 4:41 AM · Okular
ngraham added a comment to D7700: Show list of tags in PlacesView.

Showing the most recently used or most popular tags presents a usability issue since the list will be constantly changing and re-ordering itself, which impedes muscle memory and makes the software seem unstable. I would prefer that it show the tags in the order that they are created, and allow them to be re-ordered as normal--including from the "show more tags" view.

Wed, Sep 13, 3:59 AM · Dolphin

Mon, Sep 11

ngraham retitled D7776: Add "Release Year" column to Dolphin's Details view from Add "Release Year" to Dolphin information sidebar to Add "Release Year" column to Dolphin's Details view.
Mon, Sep 11, 9:13 PM
ngraham closed D7755: Add "Release Year" to Dolphin information sidebar.

All right, I will do so, and submit my own revision: https://phabricator.kde.org/D7776

Mon, Sep 11, 9:12 PM · Dolphin
ngraham added a reviewer for D7776: Add "Release Year" column to Dolphin's Details view: alexeymin.
Mon, Sep 11, 9:12 PM
ngraham created D7776: Add "Release Year" column to Dolphin's Details view.
Mon, Sep 11, 9:11 PM
ngraham updated the diff for D7763: Add Bitrate to Dolphin's Additional Information.

Rebased the diff from git master, so hopefully it will apply cleanly now.

Mon, Sep 11, 8:59 PM · Dolphin
ngraham added a comment to D7755: Add "Release Year" to Dolphin information sidebar.

I'm willing to take over ownership of the patch. I already had literally this exact diff waiting in the wings anyway in case nobody responded to https://www.reddit.com/r/kde/comments/6y3k45/ever_wanted_to_contribute_to_kde_heres_an_easy/

Mon, Sep 11, 8:54 PM · Dolphin
ngraham added a comment to D7700: Show list of tags in PlacesView.

Colored tags would be ideal. Agreed that it's out of scope for this patch, but good to work towards nonetheless.

Mon, Sep 11, 4:12 PM · Dolphin
ngraham added a comment to D7700: Show list of tags in PlacesView.

FWIW, Apple's Finder already has this exact feature, and shows 7 tags by default, plus an "All Tags" item below them.

Mon, Sep 11, 2:56 PM · Dolphin
ngraham updated the diff for D7763: Add Bitrate to Dolphin's Additional Information.

Used the appropriate metricated and internationalized bitrate converter. Thanks, @emmanuelp and @alexeymin!

Mon, Sep 11, 2:37 PM · Dolphin
ngraham added a comment to D7690: Add "Open in New Window to Dolphin's places context menu.

Thank you! I don't have commit access, so someone else will need to do it, I'm afraid.

Mon, Sep 11, 2:21 PM · Dolphin
ngraham updated the summary of D7700: Show list of tags in PlacesView.
Mon, Sep 11, 2:11 PM · Dolphin
ngraham added a comment to D7668: Add Tags browser places item to Dolphin and file pickers by default if Baloo is enabled.

https://phabricator.kde.org/D7700 is definitely better. My patch here was just a quick-and-dirty way to get it quickly. We can close mine if that gets committed.

Mon, Sep 11, 2:09 PM · Frameworks, Dolphin
ngraham added a comment to D7440: Turn on Dolphin icon previews by default.

Agreed. I haven't forgotten about this FYI.

Mon, Sep 11, 2:06 PM · Dolphin
ngraham updated the test plan for D7763: Add Bitrate to Dolphin's Additional Information.
Mon, Sep 11, 4:11 AM · Dolphin
ngraham updated the diff for D7763: Add Bitrate to Dolphin's Additional Information.

Made sure the bitrate is a float, not an int

Mon, Sep 11, 4:08 AM · Dolphin
ngraham added a reviewer for D7668: Add Tags browser places item to Dolphin and file pickers by default if Baloo is enabled: emmanuelp.

Any remaining objections?

Mon, Sep 11, 2:57 AM · Frameworks, Dolphin
ngraham added reviewers for D7446: Add a Recent Documents places item to Dolphin and file pickers by default: dfaure, emmanuelp.
Mon, Sep 11, 2:56 AM · Frameworks, Dolphin
ngraham created D7763: Add Bitrate to Dolphin's Additional Information.
Mon, Sep 11, 2:54 AM · Dolphin
ngraham updated the summary of D7755: Add "Release Year" to Dolphin information sidebar.
Mon, Sep 11, 2:28 AM · Dolphin
ngraham added a comment to D7690: Add "Open in New Window to Dolphin's places context menu.

@emmanuelp Does this look commitable now?

Mon, Sep 11, 2:08 AM · Dolphin
ngraham accepted D7755: Add "Release Year" to Dolphin information sidebar.
Mon, Sep 11, 2:04 AM · Dolphin
ngraham accepted D7755: Add "Release Year" to Dolphin information sidebar.
Mon, Sep 11, 2:03 AM · Dolphin

Sun, Sep 10

ngraham accepted D7731: Gwenview: suppressed unused interface variables compiler warnings..
Sun, Sep 10, 7:53 PM
ngraham accepted D7317: Proposed patch to fix Bug 383520: Gwenview Importer doesn't use date/time from EXIF.
Sun, Sep 10, 2:07 PM

Sat, Sep 9

ngraham added reviewers for D7317: Proposed patch to fix Bug 383520: Gwenview Importer doesn't use date/time from EXIF: KDE Applications, aacid, broulik.
Sat, Sep 9, 9:06 PM
ngraham created D7746: Give Gwenview's Save As dialog a proper title: "Save Image".
Sat, Sep 9, 2:16 AM

Fri, Sep 8

ngraham added a reviewer for D7743: Always show image rotation buttons in Image Operations sidebar when other operations are available: VDG.

ADDING VDG. I think it still makes sense to keep them in the toolbar as well, since there are many times when you might not have the Operations panel open but still want to rotate an image (it's one of the most basic operations).

Fri, Sep 8, 11:15 PM
ngraham added a comment to D7739: Add 16:9 crop preset to Gwenview.

Thanks.
Use BUG: 123456 (not the entire URL) in the commit message.

Fri, Sep 8, 11:10 PM
ngraham updated the summary of D7739: Add 16:9 crop preset to Gwenview.
Fri, Sep 8, 11:10 PM