Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421826446
I can understand your point but first, my commit message is exact that, what
the commit means. Maybe it is only a wording for sure and I know
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421802077
Done.
This is an automated message from the Apache Git Service.
To respond t
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421637536
Hm, ok. I thoght they will merged together. The Message too. But this should
Not be a Problem now. Now I know it and will make it better next
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421619118
Got it.
This is an automated message from the Apache Git Service.
To respond
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421616981
It doesn't make sense to change it now to LF and amend the commit, right? Do
I have to do it again? I mean applying the patch and fix the lin
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421616860
Hm, I closed the file and reopened it, now it is CRLF, will fix it.
This is
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421616579
This is what I said, I already checked all files, expecially SplitAction. I
change the line endings from CRLF to Unix LF before I applied the
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-421601294
So now, everyhing should be fine now. I renamed the old branch, made a clean
one, updated all branches that is needed, applied the patch, cha
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-394144054
Will try that. Thx.
This is an automated message from the Apache Git Service
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-394090220
@junichi11 Sry, I missed the point with the header. Will fix it. Thx. Yeah
maybe I need some help with that, what is not working out. I rebas
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-394086915
@emilianbold thx for your opinion and I really know what you mean. I will be
more strict in the next PRs and will keep your opinion in mind a
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-394086915
@emilianbold thx for your opinion and I really know what I mean. I will be
more strict in the next PRs and will keep your opinion in mind and
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-393828144
But in my opinion, this shouldn't be so bad to merge it now. It is as it is,
but the concept was to make is simple and only do, what needed.
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-393827803
I thought I only added that, what emi suggested. So after this, the merge
was kind of "broken" so I tried to fix it manually.
--
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-393158077
@emilianbold can you check it again please?
This is an automated message fro
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-391383476
I fixed most of the requested changes now.
This is an automated message from
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-388706086
Hm, maybe there was smth wrong with the merge :/. Sry about that. Will
change and test it later. Thx @emilianbold
-
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-388615560
So I will wait until next friday and will merge, if it is allowed(?) Please
leave a last comment thx :)
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-383603129
So now I tested and merged the changes from @emilianbold PR to my PR. Hope
now everything is fine.
-
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-379653684
@emilianbold ok I understand your point. Maybe I not finished my thinking
about the refactoring part. Anyway, I'm fine with merging your PR i
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-378832443
@geertjanw what is the preferred way, doing the stuff like @emilianbold or
is my good enough? I don't know the answer. Sure, I'm fine with it
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-378494797
@emilianbold sure your approach should work too. As I said, I wanted to
refactor the code a bit too.
---
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-378495578
But I'm also fine with your code.
This is an automated message from the Apac
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-378494797
@emilianbold as I understand it right, you wanted to leave the existing code
as it is, right? Only adding those 3 missing actions? When I und
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-378494239
The reason for changing was, I changed the code anyway so why not doing a
bit of refactoring like decouple them into separate action classes?
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-377896314
@geertjanw right. We don't see any other problems with that stuff. Maybe an
other can test it before, if someone wants it. But from our pov,
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-377686391
@sdedic markiewb and me had a coding session yesterday. Big thx to him. He
helped me a lot with other stuff and with this PR. So in general w
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-358641864
Yeah so the actions, which are added manually via `new` are for the
contextmenu of an opened tab
![image](https://user-images.githubuserc
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-358611545
@sdedic thx for reviewing, I will have a look later.
This is an automated me
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-358611486
@dtrebbien thx for the explanation, the refactoring was needed to added the
ActionIds, I tried it before I refactored the code, because that
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-355777327
Can someone have a look now, please? :)
This is an automated message from th
Chris2011 commented on issue #3: NETBEANS-59 - Document split actions
URL: https://github.com/apache/incubator-netbeans/pull/3#issuecomment-353344732
Please see my latest commits, I splitted my one commit into two as
requested. Thx.
-
32 matches
Mail list logo