jnoack (Joshua Noack)
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
Feb 26 2018, 2:00 AM (324 w, 13 h)
Availability
Available

Recent Activity

May 14 2018

jnoack added a comment to D12879: Added a page about the usage of placeholders.

Shouldn't the screenshots show english text for consistency?

May 14 2018, 6:45 PM · KDE Human Interface Guidelines

Apr 28 2018

jnoack added a comment to D12508: Make KMessageWidget match Kirigami inlineMessage's style.

Imo KMessageWidget should not rely on being placed where there's enough contrast. The transparency just doesn't work in the case of Konsole.
Either Konsole needs to move the widget outside the terminal painting area which looks complicated to achieve or the message widget should provide some API
which apps can use in cases where the background has not enough contrast.

Apr 28 2018, 11:03 AM · Frameworks

Mar 16 2018

jnoack added a comment to D11377: Expose more default window management shortcuts.

Big +1

Mar 16 2018, 5:01 AM · KWin

Mar 15 2018

jnoack added a comment to D11258: Default to show tab bar only when needed.

This change makes D10983 more relevant.

Mar 15 2018, 4:20 PM · Konsole
jnoack added a comment to D11341: Disable menu actions when read-only/locked.

Mar 15 2018, 3:45 PM · Konsole
jnoack updated the diff for D11341: Disable menu actions when read-only/locked.

Don't hide disabled actions

Mar 15 2018, 3:40 PM · Konsole

Mar 14 2018

jnoack added a reviewer for D11341: Disable menu actions when read-only/locked: Konsole.
Mar 14 2018, 9:55 PM · Konsole
jnoack requested review of D11341: Disable menu actions when read-only/locked.
Mar 14 2018, 9:54 PM · Konsole
jnoack added a comment to T7914: Use a prettier background for the lock & login screens.

I can't help me but I have some trouble to unterstand the privacy issue which comes up every time when such a topic gets discussed.
Okay I can imagine ehy it can be a privacy issue but then it would mean thaht the the background itself is one. So lets say I am sitting with my notebook on some public place like the airport und someone looks over my shoulder then he other peobple walking by can see my wallpaper too (if there are no maximized windows of course).
So privacy issue = yes but not only a problem of the login/lock-screen.

Mar 14 2018, 5:02 AM · VDG

Mar 11 2018

jnoack updated the summary of D11241: Use default close button of KMessageWidget.
Mar 11 2018, 6:23 PM · Konsole
jnoack requested review of D11241: Use default close button of KMessageWidget.
Mar 11 2018, 6:22 PM · Konsole
jnoack added a comment to D10935: Konsole: Use KMessageWidget in TerminalDisplay.

@ach Thanks for bringing this up. Found a way better approach. Will create a new revision for it.

Mar 11 2018, 6:05 PM · Konsole
jnoack added a comment to D10935: Konsole: Use KMessageWidget in TerminalDisplay.
In D10935#223346, @ach wrote:

In case it's not hidden, but destroyed, create a new KMessageWidget?
Or maybe the 'close' button can be remapped to trigger a hide instead of destroy?

Mar 11 2018, 5:44 PM · Konsole
jnoack added a comment to D10935: Konsole: Use KMessageWidget in TerminalDisplay.
In D10935#223341, @ach wrote:

Other apps (dolphin, kmail) use the 'cross on red bg' button in the upper right corner to close the msg.

I think that is much more intuitive and consistent (and shorter) than having a 'close' URL in the ever changing text formulations like 'Please click _here_' or '_Dismiss_' or ...

If you want I can submit a proper bug for this.

Mar 11 2018, 5:01 PM · Konsole

Mar 9 2018

jnoack requested review of D11196: Keep suspended mode message after tab detach.
Mar 9 2018, 10:33 PM · Konsole

Mar 7 2018

jnoack added a comment to D10935: Konsole: Use KMessageWidget in TerminalDisplay.

If the "output has been suspended" is active and that tab is dnd to create another window, the kmessage is not shown and it seems to take an extra Ctrl+q to get the tab to resume.

Mar 7 2018, 2:24 PM · Konsole

Mar 6 2018

jnoack added a comment to D10935: Konsole: Use KMessageWidget in TerminalDisplay.

Can you redo this patch so it apples cleanly to master?

Mar 6 2018, 7:32 PM · Konsole
jnoack updated the diff for D10935: Konsole: Use KMessageWidget in TerminalDisplay.

Rebase on master

Mar 6 2018, 7:29 PM · Konsole

Mar 4 2018

jnoack updated the diff for D10862: Konsole: Add read-only mode.

Explicit nullptr check

Mar 4 2018, 9:19 PM · Konsole
jnoack added a comment to D10862: Konsole: Add read-only mode.

Thanks, overall I don't see any major issues.

There are some issues below which I don't think should hold up committing this. Perhaps later you could work on them if you're interested.

  1. dbus methods still work on locked tabs: example qdbus $KONSOLE_DBUS_SERVICE /Sessions/3 sendText hi
  2. Certain menus should be disabled on locked tabs: (right click menu) Switch Profile, * Scrollback, Rename(?). Split View..... and more...
Mar 4 2018, 9:15 PM · Konsole
jnoack updated the diff for D10935: Konsole: Use KMessageWidget in TerminalDisplay.

Removed unrelated code

Mar 4 2018, 8:48 PM · Konsole
jnoack updated the diff for D10935: Konsole: Use KMessageWidget in TerminalDisplay.

@ngraham No, I don't think so. This one here doesn't work without the other one.

Mar 4 2018, 1:14 PM · Konsole
jnoack updated the diff for D10862: Konsole: Add read-only mode.
  • Read-only flag is now part of the Session and thus moving / detaching a tab won't remove it
  • Change tab icon when read-only
  • Add action to tab context menu
  • Bumped rc version
Mar 4 2018, 12:15 PM · Konsole
jnoack added a comment to D10983: Konsole: Fix invisible detached view content.

What version are you using? It works here showing the prompt.

Mar 4 2018, 9:03 AM · Konsole

Mar 3 2018

jnoack requested review of D10983: Konsole: Fix invisible detached view content.
Mar 3 2018, 1:04 AM · Konsole

Mar 1 2018

jnoack added a comment to D10862: Konsole: Add read-only mode.
  1. You have to increase the *rc version or the new files don't get installed

Just bumping is enough, right. Noted.

Mar 1 2018, 1:07 PM · Konsole
jnoack requested review of D10935: Konsole: Use KMessageWidget in TerminalDisplay.
Mar 1 2018, 1:39 AM · Konsole
jnoack added a dependent revision for D10862: Konsole: Add read-only mode: D10935: Konsole: Use KMessageWidget in TerminalDisplay.
Mar 1 2018, 1:39 AM · Konsole

Feb 28 2018

jnoack added a comment to D10862: Konsole: Add read-only mode.

With the move to KMessageWidget, let's make sure we don't regress anything: in those screenshots the widget is overlapping the scrollbar and presumably the top few lines of text too. We'll need to fix those issues.

Feb 28 2018, 2:35 PM · Konsole
jnoack added a comment to D10862: Konsole: Add read-only mode.

I have code that changes that label to a kmessagewidget; the changes were so small that I haven't bothered committing.

And you'll have to allow the possibility of having both visible at the same time.

I'll look at the code later this week

Feb 28 2018, 1:09 PM · Konsole

Feb 27 2018

jnoack planned changes to D10862: Konsole: Add read-only mode.

Putting on my VDG hat: we definitely need a visible UI during read-only mode that tells you that the terminal is read-only, or else there is a near-100% chance that people will file bugs after they turn on read-only mode by accident or forget that it's on.

Konsole already shows a yellow-colored contextual bar that appears near the top of the window in various other contexts; maybe we could re-use that here?

Feb 27 2018, 11:54 PM · Konsole

Feb 26 2018

jnoack added a comment to D10878: Changed default context menu entries on the desktop.

I agree, the context menu looks really convoluted. But what happens when the cashew is disabled and the desktop is empty? There would be no way to add widgets.

Feb 26 2018, 11:38 PM · Plasma
jnoack requested review of D10862: Konsole: Add read-only mode.
Feb 26 2018, 2:05 PM · Konsole