Opened 15 years ago

Closed 13 years ago

#4244 closed bug (fixed)

Can't remove pulse replicants.

Reported by: kvdman Owned by: stpere
Priority: normal Milestone: R1
Component: Applications/Pulse Version: R1/alpha1
Keywords: Cc:
Blocked By: Blocking:
Platform: x86

Description

On dual processor machines (or greater), when you start Pulse and drag out a replicant for the processor to the desktop, there's no way to remove it besides deleting tracker_shelf and restarting.

Attachments (1)

screenshot1.png (13.7 KB ) - added by piegod 15 years ago.
Not so useful... but attached anyway.

Download all attachments as: .zip

Change History (19)

by piegod, 15 years ago

Attachment: screenshot1.png added

Not so useful... but attached anyway.

comment:1 by piegod, 15 years ago

Platform: Allx86
Version: R1/pre-alpha1R1/alpha1

Can drag a replicant into the deskbar, but cannot remove it. Cannot right click, does the same as left clicking (does it 'disable' a CPU core? not sure of the effect :).

comment:2 by umccullough, 15 years ago

Shouldn't this be considered a bug?

comment:3 by axeld, 15 years ago

Type: enhancementbug

Indeed.

comment:4 by stpere, 15 years ago

I could fix it quite easily, but it involves having the dragger icon into the green square. It's not really pretty but if nobody finds a better solution, I could submit the correction.

comment:5 by humdinger, 15 years ago

I have of course no idea how it's implemented and if my idea is at all possible...
Right now, the CPU buttons are exactly the size of the green buttons. Could their views be enlarged with a transparent background to have enough room for the Replicant dragger? Like this:

Now:  _______
      |     |
      |  1  |
      |_____|
             O
Then: _______
      |     |
      |  1  |
      |_____|
      |////O|

"O" being the dragger, "/" the transparent part of the enlarged view.

comment:6 by stippi, 15 years ago

Yes, this is possible now. One has to use code similar to ActivityMonitor or DeskCack in order for it to work.

comment:7 by stpere, 15 years ago

Owner: changed from axeld to stpere
Status: newin-progress

comment:8 by stpere, 15 years ago

Resolution: fixed
Status: in-progressclosed

Hi, should be fixed in hrev35110. Comments welcome.

comment:9 by humdinger, 15 years ago

Looks like you forgot to include string.h in CPUButton.h.
After replicating, the CPU number isn't quite centered in its button. I did like the former, smaller buttons better, but I'll get used to it. :) Thanks!

comment:10 by humdinger, 15 years ago

Oh, and the About... menu of the Replicant says "CPUButton" instead of "Pulse". :)

in reply to:  10 comment:11 by stpere, 15 years ago

Replying to humdinger:

Oh, and the About... menu of the Replicant says "CPUButton" instead of "Pulse". :)

That point is fixed in hrev35116. Thanks!

in reply to:  9 ; comment:12 by stpere, 15 years ago

Replying to humdinger:

Looks like you forgot to include string.h in CPUButton.h.
After replicating, the CPU number isn't quite centered in its button. I did like the former, smaller buttons better, but I'll get used to it. :) Thanks!

For the size of the replicant (when used on the desktop for example), I could make the dragger a little bit closer to the button, a bit like it is on deskcalc, where it overlaps the button's border.. do you think it would be better?

in reply to:  12 comment:13 by humdinger, 15 years ago

Replying to stpere:

For the size of the replicant (when used on the desktop for example), I could make the dragger a little bit closer to the button, a bit like it is on deskcalc, where it overlaps the button's border.. do you think it would be better?

The more compact it gets the better. But only if it isn't too much work. It's really only a small issue. You've fixed the important thing, having the Replicant removable. :)

comment:14 by stpere, 15 years ago

Could you try hrev35138? Thanks

comment:15 by humdinger, 15 years ago

Very nice! Thanks Philippe!

comment:16 by rq, 13 years ago

Resolution: fixed
Status: closedreopened

It looks like this bug is valid again. I just dragged two replicants of CPU buttons into Deskbar, and cannot remove them now. I'm using hrev43238

comment:17 by diver, 13 years ago

Component: ApplicationsApplications/Pulse

comment:18 by stpere, 13 years ago

Resolution: fixed
Status: reopenedclosed

fixed in hrev44243.

Note: See TracTickets for help on using tickets.