Opened 13 years ago
Closed 8 years ago
#6834 closed enhancement (fixed)
add spinner widget [PATCH]
| Reported by: | Disreali | Owned by: | jscipione |
|---|---|---|---|
| Priority: | normal | Milestone: | Unscheduled |
| Component: | Kits/Interface Kit | Version: | R1/Development |
| Keywords: | spinner | Cc: | |
| Blocked By: | Blocking: | #8056, #8057 | |
| Platform: | All |
Description
A spinner widget would be very useful for Haiku.
Libwalter includes a spinner http://dev.osdrawer.net/projects/libwalter
Attachments (2)
Change History (15)
comment:1 by , 13 years ago
| Component: | - General → Kits/Interface Kit |
|---|---|
| Keywords: | spinner added |
| Owner: | changed from to |
| Summary: | add spinner widget → add spinner widget [PATCH] |
by , 13 years ago
| Attachment: | add_spinner.diff added |
|---|
comment:2 by , 13 years ago
| patch: | 0 → 1 |
|---|
comment:3 by , 13 years ago
I deduced the jamfile adjustment needed, and created an updated patch. I did a fresh build (jam -qa @alpha-raw) and there was no error given about adding the spinner files.
*PLEASE REVIEW* I can only assume that the code works as intended.
by , 13 years ago
| Attachment: | add_spinner_02.diff added |
|---|
comment:4 by , 12 years ago
Here is another patch that could benefit from dev review at the upcoming BG.
I do not know if the code actually does what is supposed to, as I mention before, I just copied it from libWalter.
follow-up: 6 comment:5 by , 12 years ago
I agree this could be useful but if anything this should start in the shared kit or otherwise somewhere private.
Generally we need to be careful about adding new classes like this to the Interface Kit, since it may need some API tweaking (and in this case definitely some object padding) before it can be deemed worthy of being public and therefore supported by Haiku for who knows how long.
If we can find a good use for this class in a few places in Haiku, I could see it getting added faster (though like I said, probably not to the Interface Kit first.)
Also please mark old patches obsolete when you update them.
follow-up: 7 comment:6 by , 12 years ago
| Blocking: | 8056, 8057 added |
|---|
Replying to leavengood:
If we can find a good use for this class in a few places in Haiku, I could see it getting added faster (though like I said, probably not to the Interface Kit first.)
Tickets #8056 and #8057 were created for that reason.
Also please mark old patches obsolete when you update them.
Sorry, I've not been successful in obsoleting anything on trac.
follow-up: 8 comment:7 by , 12 years ago
Replying to Disreali:
OK great, those are indeed good cases for using the spinner. I'll add this ticket and those to my personal TODO list for sometime in the next few months. I won't take ownership since I may not get to it for a while, and maybe someone else can do it in the meantime.
Sorry, I've not been successful in obsoleting anything on trac.
OK, I guess only users with certain permissions (maybe just admins, I'm not sure) can mark obsolete patches. Though it is weird you cannot mark something you uploaded as obsolete.
follow-up: 9 comment:8 by , 12 years ago
Replying to leavengood:
Replying to Disreali:
OK great, those are indeed good cases for using the spinner. I'll add this ticket and those to my personal TODO list for sometime in the next few months. I won't take ownership since I may not get to it for a while, and maybe someone else can do it in the meantime.
Sorry, I've not been successful in obsoleting anything on trac.
OK, I guess only users with certain permissions (maybe just admins, I'm not sure) can mark obsolete patches. Though it is weird you cannot mark something you uploaded as obsolete.
Please file a bug for that! I will have to look at that because it is not right.
comment:9 by , 12 years ago
comment:10 by , 9 years ago
| Milestone: | R1 → Unscheduled |
|---|
comment:11 by , 9 years ago
comment:13 by , 8 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Yes we can close this one now.
Spinner widget added in hrev49568
Attaching a patch that literally copies the Spinner.cpp and Spinner.h files from LibWalter to the, hopefully, appropriate locations in the Haiku src tree.
*Please review.* I don't know if there are any jam file adjustments that need to be done.
It would be really useful to replace the sliders in the VirtualMemory pref and DriveSetup with spinners.