jgcode 321 Report post Posted October 12, 2011 Maybe make the icon just as large as the normal clear icon... But that goes against the reason for having a small compact VI i the first place I am leaning towards this one. Anyone got any comments on the VIs Jim posted? Can it be enhanced? I am thinking that this satisfies the requirements of this review and that it should be used as is. 1 Share this post Link to post Share on other sites
GregSands 77 Report post Posted October 12, 2011 But that goes against the reason for having a small compact VI i the first place I am leaning towards this one. Is the gap between the small icon and the wires intentional? It does visually indicate that errors are cleared, but at the same time looks "wrong". Share this post Link to post Share on other sites
jgcode 321 Report post Posted October 12, 2011 Is the gap between the small icon and the wires intentional? It does visually indicate that errors are cleared, but at the same time looks "wrong". No, it is not. It is talked about in the above posts tho, and we tried to minimize it. Share this post Link to post Share on other sites
Wouter 11 Report post Posted October 13, 2011 But that goes against the reason for having a small compact VI i the first place I never said that I was in favour for the small compact VI But anyway I think I would rather like to see the icon stretched out a little then having the little gabs. Share this post Link to post Share on other sites
Francois Normandin 154 Report post Posted October 13, 2011 How about this? 3 Share this post Link to post Share on other sites
asbo 156 Report post Posted October 13, 2011 Crafty! I approve. 1 Share this post Link to post Share on other sites
jgcode 321 Report post Posted October 15, 2011 How about this? Crafty! I approve. Really? I posted that CP a few posts up and didn't get anywhere near the response for François! ...lol: Yer, that was my though too from the initial CP I posted: Or we could leave it out (no Upstream Error as per the example you posted)? I am happy with that CP too. I have all the information I require. I will move this review to pending and close it in a few days. Cheers -JG Share this post Link to post Share on other sites
Francois Normandin 154 Report post Posted October 15, 2011 I think it's not about the connector pane, but rather the small "ears" I added to the icon 1 Share this post Link to post Share on other sites
jgcode 321 Report post Posted October 15, 2011 I think it's not about the connector pane, but rather the small "ears" I added to the icon Ah-hah! (Sorry, completely missed that when I looked at it!) In that case I say "brilliant" too. Share this post Link to post Share on other sites
Yair 224 Report post Posted October 16, 2011 I'm with Wouter on this - just make the icon a bit wider. If you use François's trick, you will probably have problems with matching some of the wires, because they have a specific pattern and you would need to move the VI a couple of pixels this or that way to match it. Here's a very quick example: I think that this isn't too big and could probably be made better with a few minutes of polishing. Share this post Link to post Share on other sites
Mellroth 65 Report post Posted October 18, 2011 Anyone got any comments on the VIs Jim posted? Can it be enhanced? I would move the Merge Error inside the Error case. If "error in" doesn't contain any error the output should be the "upstream error", i.e. no need to merge. /J Share this post Link to post Share on other sites
jgcode 321 Report post Posted October 19, 2011 Thanks everyone for their input - all non-functional comments and suggestions have been reviewed with respect to updating the code Jim posted. This OpenG Review is now closed. Please start a new thread to discuss new changes to this VI. Please PM me if there are any issues with this thread. 1 Share this post Link to post Share on other sites
