Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-6030

ViewQuoteItemInfo.ftl has <td></td> at the wrong place

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: Release Branch 12.04, Release Branch 13.07, Trunk
    • Fix Version/s: 14.12.01, 12.04.06, 13.07.02, 16.11.01
    • Component/s: order
    • Labels:
      None

      Description

      When a product has been added to a quote and subsequently looking at http://demo-trunk-ofbiz.apache.org/ordermgr/control/ViewQuote it shows that the quantity and sel. amount are positioned wrong (1 column to the right of where they should be).

        Activity

        Hide
        pfm.smits Pierre Smits added a comment -

        This patch fixes the issue.

        Show
        pfm.smits Pierre Smits added a comment - This patch fixes the issue.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Pierre,

        Your patch is in
        Trunk r1654175
        R14.12 r1654176
        R13.07 r1654177
        R12.04 r1654178

        Actually, I also don't like how the quote term columns are shown. If ever you think it's worth a Jira, please feel free..

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Pierre, Your patch is in Trunk r1654175 R14.12 r1654176 R13.07 r1654177 R12.04 r1654178 Actually, I also don't like how the quote term columns are shown. If ever you think it's worth a Jira, please feel free..
        Hide
        pfm.smits Pierre Smits added a comment - - edited

        Maybe you should create the issue as you stated 'I also don't like' and describe what you think it should be. After that I might consider taking it on.

        Show
        pfm.smits Pierre Smits added a comment - - edited Maybe you should create the issue as you stated 'I also don't like' and describe what you think it should be. After that I might consider taking it on.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        I changed from bug/majoir to improvement/trivial. We agreed that a bug is when things break. Here, even if things were not good, nothing was broken. Of course, I will not revert from branches, as there is nothing to fear from this change.

        Show
        jacques.le.roux Jacques Le Roux added a comment - I changed from bug/majoir to improvement/trivial. We agreed that a bug is when things break. Here, even if things were not good, nothing was broken. Of course, I will not revert from branches, as there is nothing to fear from this change.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Mmm, I just thought about it and I don't clearly see a simple solution in first thoughts. So since this is finally minor, I prefer to forget it...
        Moreover we have other cases like that, for instance in the product prices screen, and by and large I don't think it's worth to pass too much time on this.

        Show
        jacques.le.roux Jacques Le Roux added a comment - Mmm, I just thought about it and I don't clearly see a simple solution in first thoughts. So since this is finally minor, I prefer to forget it... Moreover we have other cases like that, for instance in the product prices screen, and by and large I don't think it's worth to pass too much time on this.
        Hide
        pfm.smits Pierre Smits added a comment -

        No worries.

        Show
        pfm.smits Pierre Smits added a comment - No worries.

          People

          • Assignee:
            pfm.smits Pierre Smits
            Reporter:
            pfm.smits Pierre Smits
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development