XWIKI-21452: Macros info, success, warning and error are only distinguished by colors

xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-resources/src/main/resources/flamingo/less/messages.less

@@ -2,10 +2,52 @@
 // Messages
 // --------------------------------------------------
 
-.successmessage, .errormessage, .warningmessage, .infomessage,
-span.successmessage, span.errormessage, span.warningmessage, span.infomessage, 
-span.box { // Used by: Inline boxes and messages
-  padding: floor(@font-size-base * 0.2);
+// Used by: Inline boxes
+span.successmessage, span.errormessage, span.warningmessage, span.infomessage {
+  border: none;
+}
+
+span.box, span.successmessage, span.errormessage, span.warningmessage, span.infomessage {
+  padding: 0 floor(@font-size-base * 0.2);
+  box-shadow: none;
+  & > img {
+    // Style the silk icons
+    margin: 0 .6rem 0 .2rem;
+    vertical-align: sub;
+  }
+
+  & > span.fa {
+    // Style the font awesome icons
+    margin: 0 .6rem 0 .2rem;
+  }
+}
+
+// Used by: message boxes
+div.successmessage, div.errormessage, div.warningmessage, div.infomessage {
+  display: flex;
+  gap: 2rem;
+  justify-content: left;
+  align-items: baseline;
+  padding: 2rem;
+  border: none;
+  border-left: 4px solid;
+  box-shadow: none;
+
+  & > img {
+    // Improve alignment for silk icons
+    align-self: flex-start;
+  }
+
+  & > div > .box-title,
+  & > div > .xwiki-metadata-container[data-xwiki-parameter-name="title"] {
+    font-weight: bold;
+  }
+
+  // Main message content
+  & > div > p,
+  & > div > [data-cke-display-name="$content"] {
+    margin: 0;
+  }
 }
 
 .box, .plainmessage, // Used by: Code Macro, Success Macro, etc.
@@ -27,18 +69,26 @@ fieldset.xwikimessage { // Used by: Login form, Delete messages
 
 .successmessage {
   .alert-success;
+  background-color: @state-success-bg;
+  border-color: @brand-success;
 }
 
 .errormessage {
   .alert-danger;
+  background-color: @state-danger-bg;
+  border-color: @brand-danger;
 }
 
 .warningmessage {
   .alert-warning;
+  background-color: @state-warning-bg;
+  border-color: @brand-warning;
 }
 
 .infomessage {
   .alert-info;
+  background-color: @state-info-bg;
+  border-color: @brand-primary;
 }
 
 // --------------------------------------------------

xwiki-platform-core/xwiki-platform-icon/xwiki-platform-icon-fontawesome/src/main/resources/IconThemes/FontAwesome.xml

@@ -288,7 +288,8 @@ disk = floppy-o
 drink = glass
 drive = hdd-o
 email = envelope-o
-emoticon_smile=smile-o
+emoticon_smile= smile-o
+emoticon_unhappy = frown-o
 error = exclamation-triangle
 exclamation = exclamation-circle
 eye = eye

xwiki-platform-core/xwiki-platform-invitation/xwiki-platform-invitation-test/xwiki-platform-invitation-test-docker/src/test/it/org/xwiki/invitation/test/docker/InvitationIT.java

@@ -215,7 +215,7 @@ void sendMailToTwoAddresses() throws Exception
             assertMessageValid(messageB);
 
             // Check that the page has the table and the messages.
-            assertTrue(sent.getMessageBoxContent().contains("Your message has been sent."));
+            assertTrue(sent.getMessageBoxContent().contains("Information\nYour message has been sent."));
             TableElement table = sent.getTable();
             assertEquals(3, table.numberOfRows());
             assertEquals(3, table.numberOfColumns());
@@ -299,7 +299,7 @@ void unpermittedUserCannotSendToMultipleAddresses(TestUtils setup) throws Except
             getGreenMail().waitForIncomingEmail(2000, 2);
             MimeMessage[] messages = getGreenMail().getReceivedMessages();
             assertEquals(0, messages.length, "Messages were received when they shouldn't have been sent!");
-            assertEquals("Your message could not be sent because there were no valid email addresses to send to.",
+            assertEquals("Error\nYour message could not be sent because there were no valid email addresses to send to.",
                 sent.getMessageBoxContent(),
                 "User was not shown the correct error message.");
             stopGreenMail();
@@ -321,7 +321,7 @@ void unpermittedUserCannotSendToMultipleAddresses(TestUtils setup) throws Except
             getGreenMail().waitForIncomingEmail(10000, 2);
             messages = getGreenMail().getReceivedMessages();
             assertEquals(2, messages.length, "Non admins cannot send mail to even with permission");
-            assertEquals("Your message has been sent.", sent.getMessageBoxContent(),
+            assertEquals("Information\nYour message has been sent.", sent.getMessageBoxContent(),
                 "User was not given the message that their mail was sent.");
         } finally {
             stopGreenMail();
@@ -404,7 +404,7 @@ void spamReporting(TestUtils setup, LogCaptureConfiguration logCaptureConfigurat
                 "The message by the spam reporter is not shown to the admin.");
 
             String memo = "Actually the email lottery is quite legitimate.";
-            String expectedSuccessMessage = "Invitation successfully marked as not spam. Log entry: " + memo;
+            String expectedSuccessMessage = "Information\nInvitation successfully marked as not spam. Log entry: " + memo;
             // Return their sending privilege.
             String successMessage = inspect.notSpam("Actually the email lottery is quite legitimate.");
 
@@ -634,7 +634,7 @@ void cancelInvitation(TestUtils setup) throws Exception
                 confirm.getLabel().toLowerCase());
 
             confirm.setMemo("Sorry, wrong email address.");
-            assertEquals("Invitation successfully rescinded.", confirm.confirm());
+            assertEquals("Information\nInvitation successfully rescinded.", confirm.confirm());
 
             // Now switch to guest.
             setup.forceGuestUser();
@@ -646,13 +646,13 @@ void cancelInvitation(TestUtils setup) throws Exception
             InvitationGuestActionsPage guestPage =
                 InvitationGuestActionsPage.gotoPage(htmlMessage, InvitationGuestActionsPage.Action.ACCEPT);
             assertNotEquals("", guestPage.getMessage(), "Guest was able to accept a message which had been canceled.");
-            assertEquals("We're sorry but this invitation has been rescinded." + commonPart,
+            assertEquals("Error\nWe're sorry but this invitation has been rescinded." + commonPart,
                 guestPage.getMessage());
 
             // Prove that invitation cannot be declined
             guestPage = InvitationGuestActionsPage.gotoPage(htmlMessage, InvitationGuestActionsPage.Action.DECLINE);
             assertNotEquals("", guestPage.getMessage(), "Guest was able to decline a message which had been canceled.");
-            assertEquals("This invitation has been rescinded and thus cannot be declined." + commonPart,
+            assertEquals("Error\nThis invitation has been rescinded and thus cannot be declined." + commonPart,
                 guestPage.getMessage());
 
             // Prove that the message report spam page still shows up.

xwiki-platform-core/xwiki-platform-invitation/xwiki-platform-invitation-test/xwiki-platform-invitation-test-pageobject/src/main/java/org/xwiki/invitation/test/po/InvitationFooterElement.java

@@ -70,7 +70,7 @@ public int myPendingInvitations()
         }
         String message = getDriver().findElementsWithoutWaiting(this.footer,
             By.id("my-pending-invitations")).get(0).getText().trim();
-        return Integer.parseInt(message.substring(0, message.indexOf(" pending")));
+        return Integer.parseInt(message.substring(message.indexOf("\n") + 1, message.indexOf(" pending")));
     }
 
     public int spamReports()
@@ -80,6 +80,6 @@ public int spamReports()
         }
         String message = getDriver().findElementsWithoutWaiting(this.footer,
             By.id("spam-reports")).get(0).getText().trim();
-        return Integer.parseInt(message.substring(0, message.indexOf(" reported as spam")));
+        return Integer.parseInt(message.substring(message.indexOf("\n") + 1, message.indexOf(" reported as spam")));
     }
 }

xwiki-platform-core/xwiki-platform-invitation/xwiki-platform-invitation-test/xwiki-platform-invitation-test-pageobject/src/main/java/org/xwiki/invitation/test/po/InvitationSenderPage.java

@@ -68,7 +68,7 @@ public boolean userIsSpammer()
     {
         for (WebElement error : getDriver().findElementsWithoutWaiting(By.id("invitation-permission-error"))) {
             if (error.getText().equals(
-                "A message which you sent was reported as spam and your privilege to send mail has"
+                "Error\nA message which you sent was reported as spam and your privilege to send mail has"
                     + " been suspended pending investigation, we apologize for the inconvenience.")) {
                 return true;
             }

xwiki-platform-core/xwiki-platform-realtime/xwiki-platform-realtime-wysiwyg/xwiki-platform-realtime-wysiwyg-test/xwiki-platform-realtime-wysiwyg-test-docker/src/test/it/org/xwiki/realtime/wysiwyg/test/ui/RealtimeWYSIWYGEditorIT.java

@@ -359,7 +359,7 @@ void inplaceEditableMacro(TestReference testReference, TestUtils setup)
 
         // Save and check the result.
         ViewPage viewPage = secondEditPage.clickSaveAndView();
-        assertEquals("my info message\none two three", viewPage.getContent());
+        assertEquals("Information\nmy info message\none two three", viewPage.getContent());
     }
 
     @Test

xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/pom.xml

@@ -111,6 +111,7 @@ Thus it is forbidden to depend on the HTML Macro.
     <module>xwiki-platform-rendering-macro-gallery</module>
     <module>xwiki-platform-rendering-macro-groovy</module>
     <module>xwiki-platform-rendering-macro-include</module>
+    <module>xwiki-platform-rendering-macro-message</module>
     <module>xwiki-platform-rendering-macro-python</module>
     <module>xwiki-platform-rendering-macro-script</module>
     <module>xwiki-platform-rendering-macro-toc</module>

xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-message/pom.xml

@@ -0,0 +1,48 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<!--
+ * See the NOTICE file distributed with this work for additional
+ * information regarding copyright ownership.
+ *
+ * This is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as
+ * published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * This software is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this software; if not, write to the Free
+ * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+ * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+-->
+
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+
+    <parent>
+        <groupId>org.xwiki.platform</groupId>
+        <artifactId>xwiki-platform-rendering-macros</artifactId>
+        <version>16.3.0-SNAPSHOT</version>
+    </parent>
+    <artifactId>xwiki-platform-rendering-macro-message</artifactId>
+    <name>XWiki Platform - Rendering - Macro - Message</name>
+    <description>Extends the default message macro to provide translations.</description>
+    <dependencies>
+        <dependency>
+            <groupId>org.xwiki.platform</groupId>
+            <artifactId>xwiki-platform-localization-api</artifactId>
+            <version>${project.version}</version>
+        </dependency>
+        <dependency>
+            <groupId>org.xwiki.rendering</groupId>
+            <artifactId>xwiki-rendering-macro-message</artifactId>
+            <version>${rendering.version}</version>
+        </dependency>
+    </dependencies>
+</project>

xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-message/src/main/java/org/xwiki/rendering/internal/macro/message/XWikiErrorMessageMacro.java

@@ -0,0 +1,52 @@
+/*
+ * See the NOTICE file distributed with this work for additional
+ * information regarding copyright ownership.
+ *
+ * This is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as
+ * published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * This software is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this software; if not, write to the Free
+ * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+ * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+ */
+package org.xwiki.rendering.internal.macro.message;
+
+import javax.inject.Inject;
+import javax.inject.Named;
+import javax.inject.Singleton;
+
+import org.xwiki.component.annotation.Component;
+import org.xwiki.localization.ContextualLocalizationManager;
+
+/**
+ * Displays an error message.
+ *
+ * @version $Id$
+ * @since 16.4.0RC1
+ */
+@Component
+@Named("error")
+@Singleton
+public class XWikiErrorMessageMacro extends ErrorMessageMacro
+{
+    private static String iconPrettyNameKey = "rendering.macro.message.icon.alternative.error";
+    @Inject
+    private ContextualLocalizationManager l10n;
+
+    /**
+     * Create and initialize the descriptor of the macro.
+     */
+    public XWikiErrorMessageMacro()
+    {
+        super();
+        this.iconPrettyName = l10n.getTranslation(iconPrettyNameKey).toString();
+    }
+}

xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-message/src/main/java/org/xwiki/rendering/internal/macro/message/XWikiInfoMessageMacro.java

@@ -0,0 +1,52 @@
+/*
+ * See the NOTICE file distributed with this work for additional
+ * information regarding copyright ownership.
+ *
+ * This is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as
+ * published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * This software is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this software; if not, write to the Free
+ * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+ * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+ */
+package org.xwiki.rendering.internal.macro.message;
+
+import javax.inject.Inject;
+import javax.inject.Named;
+import javax.inject.Singleton;
+
+import org.xwiki.component.annotation.Component;
+import org.xwiki.localization.ContextualLocalizationManager;
+
+/**
+ * Displays an info message.
+ *
+ * @version $Id$
+ * @since 16.4.0RC1
+ */
+@Component
+@Named("info")
+@Singleton
+public class XWikiInfoMessageMacro extends InfoMessageMacro
+{
+    private static String iconPrettyNameKey = "rendering.macro.message.icon.alternative.info";
+    @Inject
+    private ContextualLocalizationManager l10n;
+
+    /**
+     * Create and initialize the descriptor of the macro.
+     */
+    public XWikiInfoMessageMacro()
+    {
+        super();
+        this.iconPrettyName = l10n.getTranslation(iconPrettyNameKey).toString();
+    }
+}