You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hop.apache.org by "nadment (via GitHub)" <gi...@apache.org> on 2024/10/16 17:28:48 UTC
[PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
nadment opened a new pull request, #4438:
URL: https://github.com/apache/hop/pull/4438
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
Posted by "nadment (via GitHub)" <gi...@apache.org>.
nadment commented on PR #4438:
URL: https://github.com/apache/hop/pull/4438#issuecomment-2423205556
Work fine on Windows OS.
Good point, I hadn't seen this resize part of the code.
The setButtonPositions method also posed a problem because it works with the shell and not with a composite parent.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
Posted by "hansva (via GitHub)" <gi...@apache.org>.
hansva commented on PR #4438:
URL: https://github.com/apache/hop/pull/4438#issuecomment-2422001156
seems this was the cause:
```
lsResize =
event -> {
Point size = shell.getSize();
wFields.setSize(size.x - 10, size.y - 50);
wFields.table.setSize(size.x - 10, size.y - 50);
wFields.redraw();
};
shell.addListener(SWT.Resize, lsResize);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
Posted by "hansva (via GitHub)" <gi...@apache.org>.
hansva merged PR #4438:
URL: https://github.com/apache/hop/pull/4438
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
Posted by "nadment (via GitHub)" <gi...@apache.org>.
nadment commented on PR #4438:
URL: https://github.com/apache/hop/pull/4438#issuecomment-2422279558
@hansva it's possible you talk about an other issues #4441 ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
Posted by "hansva (via GitHub)" <gi...@apache.org>.
hansva commented on PR #4438:
URL: https://github.com/apache/hop/pull/4438#issuecomment-2422374411
No I am not, this listener was hidden in the text file output
https://github.com/apache/hop/pull/4438/files#diff-c785c5464a0ac8645599379fb00b7b50cb788534af69e146f2bcf4e566626154L1251-L1259
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fields tab in Text File Output transform (UI Bug) #4436 (hop)
Posted by "hansva (via GitHub)" <gi...@apache.org>.
hansva commented on PR #4438:
URL: https://github.com/apache/hop/pull/4438#issuecomment-2422010474
@nadment , could you test this on windows and then feel free to merge.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org