Fix/add missing quickorder link in mobile view

[cbratsch]

PR Type

[x] Bugfix

What Is the Current Behavior?

The quick order link is missing in phone (xs) view. This means no access to the feature for phone devices.

What Is the New Behavior?

The quick order link is visible in phone view. It is placed in the hamburger menu (see IAD).

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

AB#65435

[shauke]

• There styling in mobile view does not match the other links
  

• The current rendering logic does not work for the Sticky Header
  

• It would be nice to still handle the feature toggle check only in the lazy component and also handle the div rendering in the header-quickorder.component.html itself so one can still use only <ish-lazy-header-quickorder></ish-lazy-header-quickorder> where ever needed. The additional input is fine. Internally ng-template (see product-image.component.html could be used).

[M-Behr]

• Space between icon and quick order link in mobile hamburger menu.
• Remove quick order link from sticky header. I discussed this with @rkarl-ish. We can later add the quick order icon next to the search icon if required. Then we have to check that the categories and the icon do not overlap in different languages and devices.

[M-Behr]

I noticed that the hover effect of the quick order link is inconsistent with the other links in hamburger menu. Intuitively, I would say yours is right and all the others are incorrect.

I discussed this with @iwiederhold and your styling is correct. Maybe the other link hover styles were removed with this CSS purge thing? Can you match the other link hover styles to yours or do we make an extra ticket for that?

[M-Behr]

I noticed that the hover effect of the quick order link is inconsistent with the other links in hamburger menu. Intuitively, I would say yours is right and all the others are incorrect.

I discussed this with @iwiederhold and your styling is correct. Maybe the other link hover styles were removed with this CSS purge thing? Can you match the other link hover styles to yours or do we make an extra ticket for that?

I tracked a minor bug for this VD issue: #805

[M-Behr]

I found an additional blank character in "Quick Order". This results in the link not being in line with the other links:

[shauke]

I found an additional blank character in "Quick Order". This results in the link not being in line with the other links:

This is fixed with the latest changes.

[M-Behr]

I found an additional blank character in "Quick Order". This results in the link not being in line with the other links:

This is fixed with the latest changes.

Hmm, but I can still see it. When I remove the blank character with the browser tool, it looks fine. I also tested it with an incognito window. Does it no longer occur in your local environment? Maybe it has something to do with the review server.

[M-Behr]

Perfect 👍