changed price position after update

WPPizza – A Restaurant Plugin for WordPress Support General Support changed price position after update

Viewing 19 posts - 1 through 19 (of 19 total)
  • Author
    Posts
  • #44508
    vesaros
    Participant

      Hi Olly,

      since the last update, the article prices switched the position under the additives.
      It even overwrote the CSS settings of my child theme.

      I am not as good as I should be in CSS and I tried to figure out which changes where made, to make it happen.
      I found the notice in the wppizza.default.css but it didn’t help me to fix it.

      My style has enough space to set the price on the right side, next to the name and the additives.
      It’s curious after the update because when an article doesn’t have additives, the price stays in the correct position.

      I don’t heavy really the time after such changes to fix this, as I said some threads before.

      This is how it looks like after the update and even with the use of a child theme:
      This is how it looks like after the update and even with the use of a child theme.

      And this is how it looks like before the update:
      And this is how it looks like before the update.

      So please be kind and tell me, how to undo these changes.

      #44512
      Olly
      Admin & Mod

        the order of the actual wrapping div, ul etc elements (prices, content text, title etc ) was changed slightly to make it possible to make things fit on a screen (affects default style only) when viewed on small screen devices while adding/changing *some* of the css to make this look as it did before

        this was simply not doable before dynamically using media queries css and in mamy cases will have resulted in quite a messed up screen when using mobile devices..

        that’s just as an explanation

        with your multitude of additives for your italien salad, this might well cause an issue and i’ll look if i can account for this too.
        In the meantime though – might also be the case – are you using any custom css on those elements ? perhaps simply removing any custom css there will also “fix” it so to speak (one might be overriding the other)

        I’ll have a look here too if i can reproduce your layout and see what happens, but if you could also check any custom css you might have and let me know , that would be great

        Could you also send me the url of your site so i can have a look at the actual css that is being applied ?

        thanks

        #44513
        vesaros
        Participant
          This reply has been marked as private.
          #44514
          Olly
          Admin & Mod

            >But with the custom css it worked very fine over a year and some updates
            sure , as unless it is strictly necessary (and in this case it was unfortunately) , i do not change these kind of things

            that aside.
            i get a a ton of javascript errors on you site (you have done something like this <!--?php . that’s really not a good idea as it will still be read and parsed in all sorts of places)
            i suspect you did that to disable your custom css (as i asked , and thanks for that)
            However, looking at what we have now, i think it will be better in this case if you just leave things as you had them – including the css you were overriding – and I will see if i can come up with a line or 2 of css you can additionally add (or change) to get you back to where we were from a layout perspective

            #44515
            vesaros
            Participant

              Okay.
              due to the fact that (i’ve copied my wpppizza.style.css into the /wppizza/css folder) it normally worked on parent and child-theme this way, I’ve deactivated the child-theme and let it on the parent theme.

              Javascript errors should be done by now.

              #44516
              vesaros
              Participant

                Okay I dunno why…but somehow it looks in Browser actually pretty good (but not on child theme)
                But it’s a complete mess on Smartphones because and again I dunno why, the additives are standing now directly after the article title and not under it. My god this CCS thing gonna kill me one day. 😮

                #44521
                Olly
                Admin & Mod

                  > the additives are standing now directly after the article title
                  i’m not with you.
                  that’s the default behaviour (but you have overridden that span with display:block and whatnot which in turn pushes everything after down)

                  it also resizes quite nicely when looking at it on a mobile device. i.e title above additives above prices – as intended and in fact the whole reason why the css/elements order was changed in wppizza 3.10

                  you sure you are not looking at some sort of cached version in your browser ?

                  #44522
                  Olly
                  Admin & Mod

                    i.e you are doing

                    .wppizza-article-default .wppizza-article-additives{ display: block; /* etc */}
                    if you do/revert to
                    .wppizza-article-default .wppizza-article-additives{ display: inline-block; /* etc */}
                    additives are next to the title (as they are by default) and things dont move around all over the place

                    if you DO want the additives to be below the title (i.e keep the display:block)

                    then you could *add* this
                    .wppizza-article-default .wppizza-article-sizes { position: absolute;top: 0;right: 0;}

                    BUT MAKE SURE you reset/remove this one again below 420px (as that’s where the media query kicks in that moves things underneath each other to cater for mobile devices )

                    #44523
                    vesaros
                    Participant

                      Hi,

                      okay please take look now.
                      I’ve rolled back to a backup before the update, so that you can see, how it was and I want it to have in the future as well. 🙂

                      #44524
                      Olly
                      Admin & Mod

                        i.e when resetting (i’m writing this out here – as reference to myself too – as i may even be adding this or similar as a default to the next update )

                        
                        @media only screen and (max-width: 420px){
                        .wppizza-article-default .wppizza-article-sizes {position: unset !important;}
                        }
                        #44525
                        Olly
                        Admin & Mod

                          >I’ve rolled back to a backup before the update, so that you can see, how it was and I want it to have in the future as well
                          the thing is, i dont think you want it like that .(hear me out)
                          have a look at it on a small screen , you will see that yoru prices are *above* the title.

                          i have had quite a few reports where people ordered the wrong thing becuase they clicked on the price BELOW the title assuming – as would be pretty much expected throughout any ecommerce site – that the price below belongs to the title above it.

                          this is precisely the reason why this got changed ……

                          if you use the new version and add the below as the last of your custom css

                          
                          .wppizza-article-default .wppizza-article-sizes { position: absolute;top: 0;right: 0;}
                          @media only screen and (max-width: 420px){
                          .wppizza-article-default .wppizza-article-sizes {position: unset !important;}
                          }
                          

                          you will get the benefits from the above problem not happening on small screen devices , plus your “original” so to speak (i.e desktop/bigger screens) to be unaffected….

                          #44526
                          Olly
                          Admin & Mod

                            PS: the easiest was/is always to simply use wppizza->layout->styles->responsive
                            (which always displays the title above everything else)

                            However, I just happen to like the default layout myself (and alwasy did), but it does need a lot of css to make it work on different devices as it also depends on how many sizes there are for an item etc …
                            (again, that’s why the changes were made in 3.10 – to make it as usable as possible “out of the box” with any – or at least most – themes)

                            …………

                            yes, i try to keep these relatively drastic changes to an absolute minimum, but it sometimes cannot be helped (to make things better for all) and if I do, I’m happy to hear about (and solve if at all possible) any issues

                            (Just felt the need to say this, lol)

                            #44527
                            Olly
                            Admin & Mod

                              coming back to the added css i suggested , you might even want to add this
                              float: none !important;
                              so the whole thing would read like this

                              
                              .wppizza-article-default .wppizza-article-sizes { position: absolute;top: 0;right: 0;}
                              @media only screen and (max-width: 420px){
                              .wppizza-article-default .wppizza-article-sizes {position: unset !important;float: none !important;}
                              }
                              

                              that will keep th eprices centered on small screen devices (if you want that). obviously , you can tweak this as you wish

                              #44528
                              vesaros
                              Participant

                                Gnah, it maybe looks perfect but it’s not.
                                Because it should be:

                                Article Name…….. Price
                                Additive………… Size

                                And the article name should be written into lines when it has 2 words which are not fitting into the line.
                                I dunno if I can explain it correctly. 🙁

                                #44547
                                vesaros
                                Participant

                                  Okay,

                                  I checked it now on a lot of Smartphones.
                                  I dunno why my Phone is the only one which shows a completely another style. (w/o cache)
                                  So at the moment, it looks fine for me and the customers.

                                  So at this point, thank you very much. 🙂

                                  #44548
                                  Olly
                                  Admin & Mod

                                    generally that all sounds good.
                                    just out of interest though, what phone are you using there you have an issue with ?

                                    #44553
                                    vesaros
                                    Participant
                                      This reply has been marked as private.
                                      #44554
                                      Olly
                                      Admin & Mod

                                        it really all depends on the amount of content you have and the device used (i.e ratio / width the screen has)
                                        i dont know anything about the Mate20 Pro.
                                        However ,having played with a few things, it seems to me it has a width of about 430 or so

                                        i can see 2 options.
                                        a) the media queries in the original css kick in at 420 i.e it’s set to do

                                        @media only screen and (max-width: 420px){/* css stuff */}
                                        you could change this to 440 for example (or something around that number). i.e
                                        @media only screen and (max-width: 440px){/* css stuff */}
                                        and the whole reshuffeling of things will occur earlier

                                        another option would be to restrict the width that the title can take up , before it starts to linebreak
                                        so, something like

                                        @media only screen and (max-width: 440px) and (min-width: 420px) {.wppizza-article-h2{display:inline-block; max-width:120px}}

                                        or similar – you’ll have to play with it a bit and or target only the items/categories where you need it

                                        (or – as mentioned previously – use the “responsive style” layout to start off with which will eliminate a lot of this messing around. But I understand that you might not want to do that of course. )

                                        PS: I would also suggest you add the float: none !important; i referred to earlier in this thread to center things under 420
                                        but that’s up to you (if you dont like it centered that also just fine of course)

                                        #44555
                                        vesaros
                                        Participant

                                          I will test it out in a few days and will report you how it worked. 🙂

                                        Viewing 19 posts - 1 through 19 (of 19 total)
                                        • The topic ‘changed price position after update’ is closed to new replies.