NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar Ruud68
Ruud68
18 Sep 2020

Pull Request for Issue #30667

Summary of Changes

Add flex wrap to enable wrap on tabs

Testing Instructions

  • apply pr,
  • npm run build:css

Actual result BEFORE applying this Pull Request

see issue #30667

Expected result AFTER applying this Pull Request

see issue #30667

Documentation Changes Required

nope

avatar Ruud68 Ruud68 - open - 18 Sep 2020
avatar Ruud68 Ruud68 - change - 18 Sep 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Sep 2020
Category Administration Templates (admin) NPM Change
avatar sandewt sandewt - test_item - 18 Sep 2020 - Tested successfully
avatar sandewt
sandewt - comment - 18 Sep 2020

I have tested this item successfully on 7778840

Joomla! 4.0.0-beta5-dev Development [ Mañana ] 15-September-2020 19:15 GMT


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30675.

avatar Ruud68
Ruud68 - comment - 18 Sep 2020

Thanks @sandewt for testing!

avatar richard67
richard67 - comment - 18 Sep 2020

@chmst @wilsonge Could you test this PR with the laptops you have used for reproducing issue #28799 ? To me it seems other issues than the tabs, e.g. the width of the toolbar and header area either have been solved elsewhere meanwhile, or they are magically solved with this PR, too, or my 2 screens are not suitable to reproduce the other issues.

avatar richard67 richard67 - test_item - 18 Sep 2020 - Tested successfully
avatar richard67
richard67 - comment - 18 Sep 2020

I have tested this item successfully on 7778840

Not sure if it is the best solution, but it is a solution which works, and for me it's ok.

Hoping for more opinions.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30675.

avatar richard67 richard67 - change - 18 Sep 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 18 Sep 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30675.

avatar richard67 richard67 - change - 18 Sep 2020
Labels Added: ? ? NPM Resource Changed
avatar richard67
richard67 - comment - 18 Sep 2020

2 good tests, so RTC.

But I'd still like to know other opinions and if it has fixed issue #28799 completely or not.

avatar Ruud68
Ruud68 - comment - 18 Sep 2020

@richard67 thanks for testing and RTC :)

This is the default way bootstrap 4 implements tabs, see here: https://getbootstrap.com/docs/4.5/components/navs/#tabs

avatar chmst
chmst - comment - 18 Sep 2020

In any case it is better than before and resolves the issue with horizontal scrolling. So RTC is good for me.

avatar wilsonge
wilsonge - comment - 18 Sep 2020

I mean it's ugly as hell but it wraps. Which is more than it does now

avatar wilsonge wilsonge - change - 18 Sep 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-18 20:50:42
Closed_By wilsonge
Labels
avatar wilsonge wilsonge - close - 18 Sep 2020
avatar wilsonge wilsonge - merge - 18 Sep 2020
avatar wilsonge
wilsonge - comment - 18 Sep 2020

Thanks!

Add a Comment

Login with GitHub to post a comment