M P

Hey @m155y5,

yeah I got it solved with some elbow grease.

Just to make sure, you do understand that by sending emails I didn’t ask for you are spamming me. And don’t give me that “you can subscribe anytime” bullshit. It applies only if I have actually opted-in to receive emails from you.

If you want a review about the plugin, here goes:
– Too much buy this buy that buy premium crap everywhere
– The whole way-too-busy user interface is unnecessary for a plugin that generates a robots.txt

It’s WIP page so I don’t have a link for you.

But I did solve the problem with some hackety hack CSS:

.fancybox-slide {
    transform: translate(0,0) !important;
}

Hey Nastia.

Well, the fact that I have to edit .po translation files to change a button text is quite telling. The plugin is suppose to be all about hustling, getting opt-ins and so on yet you can’t get even do basic conversion optimization.

So about the code quality. I was using jQuery to modify this crap on-the-fly to get it even remotely resemble a converting popup, and to prevent the thing opening all the time after visitor has already opted-in.

Here’s few nice things you can try:
– Try to change input field’s placeholder. Oops you can’t because it’s coded by somebody who doesn’t understand basic CSS positioning so the labels are on top of the fields and if you edit them, or god forbid, even try to change the font size, everything breaks because you have essentially hardcoded the positioning based on certain sizes. This was how things were done in the 90s, in 2000s we use responsive positioning.
– Try to change the button text with jQuery. You have to write a monitoring script for the DOM because you first put the template in the head with CDATA and some other completely weird shit but then you don’t actually use that template but instead you probably load it into memory when loading and then use that. And at load you generate the popup in the head but you won’t actually use that when it triggers but instead you create a new element that is the actual popup. So the monitor script has to beat the DOM to find out if there’s button with certain class and then change the text.

So yeah, let’s just say it’s not good…