U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » Developing U++ » UppHub » LedLabel new package

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: LedLabel new package [message #42483 is a reply to message #42481]

Wed, 19 March 2014 08:52

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

Nice. I have just two suggestions:

IMO,

	Image On, Off;
	bool Status;
	Label Caption;

should be private and some methods provided to change them. Besides, changing On/Off or Status should call Refresh to reflect changes.

Also, images in .iml file have wrong aliasing of borders. This means that with border other than something light-gray, borders will look ugly.

Report message to a moderator

[Message index]

		LedLabel new package By: deep on Tue, 18 March 2014 11:16
		Re: LedLabel new package By: deep on Tue, 18 March 2014 11:17
		Re: LedLabel new package By: jerson on Tue, 18 March 2014 11:27
		Re: LedLabel new package By: koldo on Tue, 18 March 2014 14:16
		Re: LedLabel new package By: deep on Tue, 18 March 2014 14:24
		Re: LedLabel new package By: deep on Tue, 18 March 2014 15:11
		Re: LedLabel new package By: Klugier on Tue, 18 March 2014 20:31
		Re: LedLabel new package By: deep on Wed, 19 March 2014 02:46
		Re: LedLabel new package By: mirek on Wed, 19 March 2014 08:52
		Re: LedLabel new package By: koldo on Wed, 19 March 2014 09:41
		Re: LedLabel new package By: deep on Wed, 19 March 2014 11:27
		Re: LedLabel new package By: deep on Fri, 18 April 2014 16:31
		Re: LedLabel new package By: deep on Sat, 19 April 2014 16:24
		Re: LedLabel new package By: BioBytes on Wed, 23 April 2014 21:20

Previous Topic:	Upp package binding for LLVM/Clang library (libclang)
Next Topic:	VLCPlayer compile only with libvlc v. 1.1.10 and later

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Fri Apr 19 22:06:20 CEST 2024

Total time taken to generate the page: 1.05820 seconds