Merge pull request #4086 from crazyserver/MOBILE-4470

MOBILE-4470 workshop: Remove changes confirmation when not editing

src/addons/block/activitymodules/components/activitymodules/activitymodules.scss

@@ -1,7 +1,7 @@
 :host {
     core-mod-icon {
-        margin: 0;
-        --padding-start: 0px;
+        --margin-vertical: 0px;
+        --margin-end: 12px;
         --module-icon-size: 24px;
     }
 }

src/addons/mod/workshop/components/assessment-strategy/assessment-strategy.ts

@@ -250,7 +250,7 @@ export class AddonModWorkshopAssessmentStrategyComponent implements OnInit, OnDe
      * @returns True if data has changed.
      */
     hasDataChanged(): boolean {
-        if (!this.assessmentStrategyLoaded || !this.workshop.strategy) {
+        if (!this.assessmentStrategyLoaded || !this.workshop.strategy || !this.edit) {
             return false;
         }
 

src/core/components/mod-icon/mod-icon.scss

@@ -16,7 +16,7 @@
 
     margin-top: var(--margin-vertical);
     margin-bottom: var(--margin-vertical);
-    @include margin-horizontal(null, var(--margin-end));
+    @include margin-horizontal(0px, var(--margin-end));
 
     border-radius: var(--icon-radius);
     @include padding(var(--padding-top), var(--padding-end), var(--padding-bottom), var(--padding-start));
@@ -77,3 +77,10 @@
     --margin-vertical: 12px;
     --margin-end: 12px;
 }
+
+:host-context(html.dark) {
+    &.version_40:not(.colorize),
+    &.version_current {
+        background-color: var(--white);
+    }
+}