Forum Replies Created
- AuthorPosts
-
>re: Some customers may get confused as they can’t choose collection.
as mentioned, I agree with you there (and there should be an option to select “self-pickup”
>re: When testing on a Microsoft Surface 2 I get this error (and it only happens on this device)
I am assuming we are talking about https://www.mahersyedali.co.uk herea) i tried this on a variety of different browsers, opera , chrome, firefox, ie 8-11, on mobiles as well as on desktops. none throw an error (on the page that is).
b) when you get that error , do you get any errors in the console (as it’s an MS thing, I supposee you have to press F12 and then click on console to see what it says – let me know if there’s anything that might throw a light on this).
c) also, check if you get anything in your debug log.
d) However, before you even do any of this, fix your errors.
you are mixing secure with nonsecure content (not always crucial, but certainly a good idea to fix) as well and have unclosed element tags (read invalid html, which can/could break a ton of things)e) you also want to check your debug log. the PHP NOTICE errors are not that vital (and can be ignored for the time being ) however , your database errors are certainly not good (to say the least).
(i did notice that there are some wppizza notices in there too. although – as mentioned – notices are pretty inconsequential, i will fix those too (provided they are not caused by something else of course)as for everything else, i believe I already addressed this above
PS: generally speaking, there are a couple of improvements that can certainly be made (as always), and I will bear those in mind and do something about them as soon as i get the chance.
I believe though that your particular idea/suggestion is not the best way to go about it
then again, that would be my job anyway, so the essence of your post is certainly something to considerif you do this (in no particular order):
a) you have just blocked every single searchengine on the planet from indexing that page (they don’t do javascript so will never click on things and therefore never get past that content – but also see g) – which would get around that )
b) if you have more than one page with menu items and you only what that sort if thing on your main menu (i.e “order online”)page, it would be pretty pointless, as many/most people that come to your site from searchengines would never get to see it in the first place
c) you cannot cache that page – ever (you might not want to , but just saying)
d)>re: but how would I set where the link goes via css.
you do not / cannot change links via css (that’s not what css is)e)>:re I know you already have this in a shortcode
I do ? not that I know of. (but maybe I am not following)f)re: I want to do this because on some devices I have tried the website on popups can be a bit troublesome
sorry, but I somewhat dispute that unless you have a slow server to start off with or there are other things wrong. (but see g) below as well)
having said that, of course I’m not perfect, so if you can show me what is apparently troublesome, I’d be happy to investigate.g) re:I want the delivery/collection choice to be made before ordering ideally.
that’s why there is the option of the popup (i assume we are talking about the delivery by postcode extension here)
instead of having a popup, one could however replace the *contents* of the page entirely dynamically, rather than having a popup – i’ll investigate , but no ETAh) i furthermore agree , that there is a case that can be made for having an option in the dropdown/selection that offers “i will pick up the order myself” or some such thing
(in fact, I am somewhat surprised no-one ever mentioned this before)>If it is mandatory to have it on the page…
put the cart on the page and set the display/css to none.
and then i suppose use
[wppizza type=’totals’]
see https://www.wp-pizza.com/topic/wppizza-shortcodes/v 4.7.1 should hopefully also now take care of those bits
seems fine now.
you do however seem to have some strange visual artefacts next to any selected/added ingredients (on the right).
that has something to do with your theme , but i’ll see if i can do something that deals with this too and forces these to disappearit’s actually the way you have designed your scrollbars.
will have to do a bit of tweaking, but it’s minor in any case and has nothing to do with functionalityfor example, the version you put back on the server will not have a “comment” before any additional comments (as the db will not have been updated to that effect), and you also have old css files….
what changes discussed did *you* do to that plugin (unless you are talking about different things entirely) ?…
in any case, the latest is 4.7 and – looking at your files/outcome there – you are somewhere in between
when i put things on your test server, i did not always update versions numbers when i changed things (as that can get tedious when testing things, before actually making it an official release so to speak), so I would have thought best thing is – to make sure we are looking at the same thing – i’ll put 4.7 on there (if you give me the ok)
ok
on a different note though, you seem to have overwritten the AI plugin on your test server again with old files ?! (or old db entries ? cant be sure)
>Not sure how to test it has worked though 😉
just look at the page source
if it sayslink rel=’stylesheet’ id=’cff-css’ href=’http://blablablablbalab/cff-style.css?ver=4.1
etc it’s fine otherwise it just says
link rel=’stylesheet’ id=’cff-css’ href=’http://blablablablbalab/cff-style.css
(same goes for js files)
that bit does:
remove_wp_ver_css_js
to spell it out
——no version numbers——a) day 1: customer comes to site , does his thing
b) day 2: plugin gets updated, javascript changes and server expects those js variables to be passed on
c) day 3: customer comes back to site, customers browser says: wait, i do not need to downlaod that javascript again, i already got it cached, let m use that oned)result=> some, more or fewer of required variables get sent to server as old js is being used
e) server response: what do you want from me ?
—– with version numbers——
a) and b) are the same
c) customer comes back to site, browser says: “hang on, that javascript file has changed , better download/use the new version”d)… we all live happily ever after…
(same goes for css by the way ,except that it won’t be fatal, just screws up your layout)
as long as you are omitting version numbers , i am not even going to try, sorry…
dunno, maybe your theme or some plugin does this…
Not quite sure what’s not a good idea…
omitting version numbers
>and let you know of any faults.
please douploaded an updated version to your test server.
should now all be fine (and include a label for textbox entries)
will do some checking tomorrow..
PS:
you test server omits version numbers in linked files (css/js) , so you might have to clear your cache/reload pages(it isn’t such a good idea to do that btw)
i guess i could add some label in front of those comments that makes it clearer that it’s a comment or something
in any case.
this will not be an issue anymore with the update which should be done tomorrow
(just need to tweak a couple more css bits)ie , NEVER have
define(‘WP_DEBUG_DISPLAY’, true);
on a production servera) can’t replicate this (as in being an error , stopping me from adding this to the cart)
b) i *can* see this in the debug log. easy enough to fix
(though a production server should never output these errors, only log them . if you output anything to ajax other than what needs to be there (warnings,errors, notices – you name it WILL fall over – hence the error_reporting(0) at the top of the ajax files as – more often than not – other plugins throw many many notices)… anyway, i digress…that’s a whole topic in itselfc) – pretty much just ignore b) . i’ll take care of those particular warning …
- AuthorPosts