Forum Replies Created
-
AuthorPosts
-
works all fine here as intended
i’d suggest you enable debug and check your consoleThis reply has been marked as private.I can actually see in the code that i deliberately omitted this option at the time.
That said, I cannot really recall why and con only think of either there being some security implications of some sort (which of course there are) or I got sidetracked with something else and never finished it off, leaving it until someone asks for it
– i.e nowlet me see what I can do in the next update – it’s not quite as trivial as one would want it to be though unfortunately.
only thing i can see here is that there is no var called
$order_table_name
there is however a constant one can use insteadWPPIZZA_TABLE_ORDERSother than that – as this’ not the full code/function/hook i cannot comment too much other than – under normal circumstances – the
$order_details['customer']array makes no distinction between cod or prepay(i.e cc) orders in that action hook (it only exists once regardless of what payment type was used)
unless something was done via some filter before of course (in which case your thank you page etc would also be missing a bunch of details though)a) php notice: inconsequential of course , but will be eliminated in the next update. thanks for mentioning it
b) without seeing the settings (and a link/site where one can see what happens) i cannot say much
9 December, 2017 at 5:16 am in reply to: Custom Error Message per Dish with Ingredients/Options #33483No, sorry. There is no provision for that I’m afraid.
(to be honest, I cannot even see under what circumstances this would be of particular benefit other than perhaps some minor and somewhat inconsequential cosmetics such as adding the item name perhaps to the message. never mind that you would have to add a multitude of individual messages, one for each item…)alternatively
wppizza->layout -> miscellaneous->Hide cart icon next to prices
use css
.wppizza-article-default .wppizza-article-size:after
is the one that displays ithad a mess around with things here for a bit
setting a limit of 300 as opposed to 420 as it is now, seems quite acceptable as well (always provided the theme itself is responsive to start off with)will be added/updated in the next update
so , just for the fun of it , i messed around with a couple of things.
and – contrary to what i expected actually –
<?php echo single_cat_title(); ?>as well as<?php echo category_description(); ?>
works also just fine from what i can see here (at least with the 2016 theme)….dunno
PS
i guess you are/were referencing the wrong file in you second paragraph there(i.e
/isladelencanto-child/wppizza/markup/loop/theme-wrapper.phpon line 18. when you meantwppizza/markup/cart/cart.container.php)assuming that’s the case, feel free to do what you want there if that works for you
must admit, i don’t know how this relates to the topic title (“Can no longer add item by name”)…?
anyway, just referring to your last reply here
1. breadcrumbs: wppizza is a custom post type a simple single_cat_title() and category_description() are unlikely to have ever worked there (though there might be some circumstances where this might have worked, but that would be more accidental then anything and pretty much them dependent i would have thought)
i would suggest you have a look here
https://wordpress.stackexchange.com/questions/204738/breadcrumbs-with-custom-post-type-without-plugin
(i might even add a function to the plugin to that effect at some point for convenience)2. all themes are different and i cannot say anything about “line18” there .
essentially , you can do what you want if this works for you, though why you would need any kind of cart thing in the loop template only you will know . of course that might have something to do with your theme and the way it’s constructed>Have a good weekend : )
you too …will post it here
and you can always override the css anyway if you want it done sooner
the one that does it is this
@media only screen and (max-width: 420px)….ok, i – mistakenly – assumed we are talking about something else (my fault)
that out of the way, there is some css in there (i.e display:none) that tries to account for the more limited space available in the cart on small-screen devices.
that said, at first glance at least , you might actually have a point that I might be overdoing things here.
Though I am quite certain that “hiding” the quantity display (as that’s what we are talking about here) was done for a reason at the time , with all the adjustments done afterwards, this might now be the wrong thing to do.Allow me to think about this for a bit (taking into considerations several setup/theme possibilities), and if it turns out that this is somewhat overkill, I’ll gladly get rid of that declaration in the next update
thinking out loud for a minute, i would think that this declaration still has a role to play, but should perhaps/probably only apply to screen/device sizes smaller than it is at the moment (which is 420)
i’ll need to play wit that is for a bit.
no promises – as i do not know yet the implications it might have – but thanks for flagging the issue neverthelesscannot see the minicart there being used/enabled ?!
(mobile view or not)this sounds more like your theme is hiding certain things in mobile view as the plugin does not do that
unless you are telling it to using the available (shortcode) attributes if adding the minicart by shortcode instead of using the widget
(i cannot say any more without a link to your site I’m afraid)>/wppizza/markup/cart/cart.container.php
>/wppizza/markup/loop/theme-wrapper.phpthose are – already – v3 files…
must admit , i don’t really understand what the problem seems to be here …sorry, but i don’t think i understand the question exactly (or actually the requirement as to why you would need this )
-
AuthorPosts