Skip to content

[4.0] House cleaning => remove obsolete div element in system message container#31195

Merged
chmst merged 2 commits intojoomla:4.0-devfrom
hans2103:feature/house-cleaning-and-remove-obsolete-div-element-in-system-message-container
Oct 22, 2020
Merged

[4.0] House cleaning => remove obsolete div element in system message container#31195
chmst merged 2 commits intojoomla:4.0-devfrom
hans2103:feature/house-cleaning-and-remove-obsolete-div-element-in-system-message-container

Conversation

@hans2103
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

This PR will remove the obsolete <div id="system-message"></div> from layout/joomla/system/message.php
This id was used in J3 plugin installer_webinstaller. It is not used in the J4 version anymore.
Therefor this element is obsolete and can be removed.
This PR will also remove the obsolete styling for this removed element.

Testing Instructions

  • Joomla 4
  • Find the login module in the sidebar, don't fill any fields and press the button login.
  • The page refreshes on /index.php/author-login with an alert

Empty password not allowed.

The HTML of this alert is:

<div id="system-message-container" aria-live="polite"><joomla-alert type="warning" dismiss="true" class="joomla-alert--show" role="alert"><button class="joomla-alert--close" aria-label="Close"><span aria-hidden="true">×</span></button><div class="alert-heading"><span class="warning"></span><span class="sr-only">Warning</span></div><div class="alert-wrapper"><div class="alert-message">Empty password not allowed.</div></div></joomla-alert></div>

Press the X to close the alert

Actual result BEFORE applying this Pull Request

You are able to close the alert.
The HTML of the closed alert is:

<div id="system-message-container" aria-live="polite"><div id="system-message"></div></div>

Expected result AFTER applying this Pull Request

You are able to close the alert.
The HTML of the closed alert is:

<div id="system-message-container" aria-live="polite"></div>

Without the obsolete div element you can properly use the css :empty trigger to remove a margin between succeeding elements. No JS needed.

Documentation Changes Required

@infograf768
Copy link
Member

I have tested this item ✅ successfully on 891ac39

OK for this.

@chmst #system-message-container needs to be added to Cassiopea css with margin-top: 0;


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

@hans2103
Copy link
Contributor Author

hans2103 commented Oct 22, 2020

@chmst #system-message-container needs to be added to Cassiopea css with margin-top: 0;

No... only #system-message-container:empty and the :first-child of the parent of #system-message-container needs a margin-top:0.
This should be done after merge of this PR

@hans2103
Copy link
Contributor Author

PR joomla/cassiopeia#194 is created to fix obsolete margin in Cassiopeia

@adj9
Copy link

adj9 commented Oct 22, 2020

I have tested this item ✅ successfully on 891ac39

OK
Schermata 2020-10-22 alle 14 38 57


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 22, 2020
@chmst chmst merged commit 04e74c8 into joomla:4.0-dev Oct 22, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Oct 22, 2020
@chmst
Copy link
Contributor

chmst commented Oct 22, 2020

Thanks!

@hans2103 hans2103 deleted the feature/house-cleaning-and-remove-obsolete-div-element-in-system-message-container branch October 22, 2020 14:04
@zero-24 zero-24 added this to the Joomla 4.0 milestone Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants