Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed the Add Overlay button and a few other bugs/warning #54

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

killerog
Copy link

@killerog killerog commented Dec 8, 2012

The console gave an error in ppLockAnimationModule.js and after a little searching it seemed that variables where decleared in the regular way instead of like "var : value," like it is supposed to be in FBL.extend.

…ormat instead of the regular variable declaration.
@photoman
Copy link

photoman commented Feb 6, 2013

It would be great to have this fix implemented, as it doesn't work in my FF18.0.2. There is no "add overlay" button as described.

@deoism
Copy link

deoism commented Feb 6, 2013

any word on this fix being implemented - would be nice - I WANT TO USE THIS PRODUCT - WHO DO I TALK TO?

@killerog killerog closed this Feb 8, 2013
@killerog killerog reopened this Feb 8, 2013
@killerog
Copy link
Author

killerog commented Feb 8, 2013

Sorry guys for the confusion, apparantly Git already added my new fixes to this request and I missed it. As this project doesn't seems to be actively maintained anymore, you can find the firefox installer here and you can view the source here
Should also help for #48 #53 #50 and probably a few more.

@photoman
Copy link

Unfortunately it still doesn't work for me. I checked it in FF on a Mac and PC. I do see the add overlay button and it did add a file - but the check box next to the file list can't be checked. Guessing that is needed to activate the overlay iamge. Also the x, y positioning or opacity doesn't incriment or change at all.

I prefer firefox, but until this gets fixed.. here's one for Chrome that seems to work pretty well: http://www.welldonecode.com/perfectpixel.html

@killerog
Copy link
Author

@photoman I have tried that and for me it seemed to work. The only thing was that if you use a small image, that it gets centered out of screen for me. Could that be your problem too?

Else could you please give me your firefox and firebug version so I can check that.

@photoman
Copy link

Using the latest version 18.0.2 in mac and pc. I can't even activate the overlay image.. the check box won't check.

@killerog
Copy link
Author

And which version of firebug?

@photoman
Copy link

PC: ver 1.11.1
Mac: ver 1.10.4

@mbuckley
Copy link
Owner

mbuckley commented Mar 9, 2013

Sorry for the lack of updates guys. I've rolled back the v2 code so we now have a working v1.8 in the master branch.

The new release (1.8) has been submitted to mozilla for approval. The add overlay is now located in the Pixel Perfect menu. See the new screenshots on the addon page (https://addons.mozilla.org/en-US/firefox/addon/pixel-perfect/?src=search) for details.

If you'd like to try it out before it is publicly available you can download the release from this url:

https://addons.mozilla.org/en-US/firefox/addon/pixel-perfect/versions/

@photoman
Copy link

Sorry.. it still doesn't work for me. Using FF 18.0.2 -- I can't click the check box next to the layer to enable the overlay I have loaded into the app. Please try again. Perhaps I'm doing something wrong? I'm using a JPEG file for my overlay.

@killerog
Copy link
Author

@photoman I accidentally ran into the same issue you seem to have when I was running with firebug detached. I'll try to reproduce it again, because I forgot to write down the error...

Yeah, I'm am idiot to forget that :)

Conflicts:
	srcExtension/chrome/content/ppLockAnimationModule.js
@mkrishtopa
Copy link

Same problem. Can add overlay, but it can't be enabled by clicking on the checkbox. FF23.0.0/Win.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants