mozilla / addons-blog

Blog content builder for AMO
Mozilla Public License 2.0
6 stars 4 forks source link

Hide header section links on Android #600

Closed willdurand closed 1 year ago

willdurand commented 1 year ago

Fixes https://github.com/mozilla/addons-blog/issues/331


We cannot do much more than fixing the header with JavaScript unfortunately, especially if we want to keep the other items for non-Android platforms.

codecov[bot] commented 1 year ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0ee3a68) 87.31% compared to head (19e50f9) 87.58%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #600 +/- ## ========================================== + Coverage 87.31% 87.58% +0.27% ========================================== Files 9 9 Lines 544 556 +12 Branches 139 143 +4 ========================================== + Hits 475 487 +12 Misses 62 62 Partials 7 7 ``` | [Files](https://app.codecov.io/gh/mozilla/addons-blog/pull/600?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mozilla) | Coverage Δ | | |---|---|---| | [src/assets/js/header.js](https://app.codecov.io/gh/mozilla/addons-blog/pull/600?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mozilla#diff-c3JjL2Fzc2V0cy9qcy9oZWFkZXIuanM=) | `100.00% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.