The Pause icon is usually not what you're interested in. The activity we perform most-frequently is "Resuming execution" which is more of a "Play" action than "Pause". We have options: 1. Always use a Play icon instead of Pause. 2. Switch between Play and Pause depending on state (checked would switch to Play, unchecked would be Pause). 3. Incorporate a single Play/Pause button that doesn't change. The |> button.
Paul suggests option 1. Always use the Play icon. I'm inclined to say yes, let's do this for this release. Later on when we have the Threads panel we'll need to be able to pause workers and the pause button may make sense for that.
Currently there is no icon for the "play/pause" button when the script is not paused on Win7. I'm not sure if this bug is meant to add the icon there, or if this bug is supposed to change the icon that is supposed to be there. See this screenshot for an example of what I'm talking about: http://screencast.com/t/jbptmYe93X
Created attachment 628675 [details] [diff] [review] Windows fix for the pause icon Confirmed the missing pause icon. This patch should fix it, although my Win VM will take a while to verify this.
(In reply to Panos Astithas [:past] from comment #5) > Created attachment 628675 [details] [diff] [review] > Windows fix for the pause icon > > Confirmed the missing pause icon. This patch should fix it, although my Win > VM will take a while to verify this. Fix verified.
this isn't really the bug for that. This is about replacing the pause icons with play icons. I'll have a patch ready (that includes windows) today.
Created attachment 628907 [details] [diff] [review] play icons ok, here it is. I left the pause icon in place figuring well want it again.
Created attachment 628929 [details] [diff] [review] play icons v1