- User Since
- Sep 4 2015, 9:41 PM (170 w, 4 d)
I'll see if I find anything after testing
ViewContainer.cpp:133:44: warning: Missing emit keyword on signal call Konsole::TabbedViewContainer::newViewWithProfileRequest
Mon, Dec 10
Nice catch thanks
This is really good - I noticed that once you detach/close views, the menus View->Close/Expand/Shrink are still active (they don't do anything when selected).
Sat, Dec 8
I've noticed a few issues
- detach split tabs maintain the same size (ie not the full window) but the size the tab was - is this expected behavior?
- close one side of the view causing the menus to not have all the menus until switching tabs
- opening a new tab on the non-focus side will not update the tab text (ie should be "src:zsh", it is ":" until you click on tab).
Fri, Dec 7
Thu, Dec 6
Wed, Dec 5
Ok let me look at this - I notice if you double-click on an empty spot in the non-focused view's tabbar, the new tab is created in the focused view. Which is a bit confusing.
w/o the Konsole reviewer, it doesn't show up on https://phabricator.kde.org/project/37/item/view/637/ which is what I use
OK so this is for when 'Use Line characters contained in in Font' is disabled. I see a big improvement then.
Of course thanks
Tue, Dec 4
Wow, that's impressive. Thanks a lot for working on this - let me look at it further.
Ok, I think I/someone read the docs and did it that way from the start - I wonder what the Qt docs thinks is a large file.
Sun, Dec 2
same issue here - can you fix?
The arc patch is moving files - can you re-upload?
Sat, Dec 1
Ok if you want - I thought you might be still working on it
Fri, Nov 30
Looks fine - thanks
Thu, Nov 29
Sun, Nov 25
Yes, I was actually surprised I managed to get it all to run. Are there official docs on how to setup an ci image and run applications? If not, it would be helpful.
Sat, Nov 24
Fri, Nov 23
Thu, Nov 22
I manage to build the docker kdeorg/ci-suse-qt59 image - and of course the tests pass - so I'm not sure how to help now. I've already disabled this test again to avoid the sysadmins having more work to do.
This throws ASAN errors
Tue, Nov 20
The change appears to be so slight as to be unnoticeable. Only when I change the 48 to 148 can I actually see changes. I would suggest following up on your comment about not just checking for pure black and then also make the change noticeable.
I think the icons are correct as-is, though their may be some other issue w/ the themes as detailed in the bug report. The 'Initial Dir' is asking for a folder so folder-open seems correct. The "User user-defined stylesheet' is asking for a file so the document-open seems OK.
OK the boxes look better w/ patch regardless
Fri, Nov 16
yea - feel free to commit any changes related to this yourself if you want.
Thanks so much for looking into this.