diff --git a/app/assets/javascripts/discourse/app/components/admin-post-menu.gjs b/app/assets/javascripts/discourse/app/components/admin-post-menu.gjs index 91edb191c3f..ca2091cbc71 100644 --- a/app/assets/javascripts/discourse/app/components/admin-post-menu.gjs +++ b/app/assets/javascripts/discourse/app/components/admin-post-menu.gjs @@ -14,6 +14,39 @@ export default class AdminPostMenu extends Component { @service store; @service adminPostMenuButtons; + get reviewUrl() { + return `/review?topic_id=${this.args.data.transformedPost.id}&status=all`; + } + + get extraButtons() { + return this.adminPostMenuButtons.callbacks + .map((callback) => { + return callback(this.args.data.transformedPost); + }) + .filter(Boolean); + } + + @action + async topicAction(actionName) { + await this.args.close(); + + try { + await this.args.data[actionName]?.(); + } catch (error) { + // eslint-disable-next-line no-console + console.error(`Unknown error while attempting \`${actionName}\`:`, error); + } + + await this.args.data.scheduleRerender(); + } + + @action + async extraAction(button) { + await this.args.close(); + await button.action(this.args.data.post); + await this.args.data.scheduleRerender(); + } + {{#if this.currentUser.staff}} @@ -205,37 +238,4 @@ export default class AdminPostMenu extends Component { {{/each}} - - get reviewUrl() { - return `/review?topic_id=${this.args.data.transformedPost.id}&status=all`; - } - - get extraButtons() { - return this.adminPostMenuButtons.callbacks - .map((callback) => { - return callback(this.args.data.transformedPost); - }) - .filter(Boolean); - } - - @action - async topicAction(actionName) { - await this.args.close(); - - try { - await this.args.data[actionName]?.(); - } catch (error) { - // eslint-disable-next-line no-console - console.error(`Unknown error while attempting \`${actionName}\`:`, error); - } - - await this.args.data.scheduleRerender(); - } - - @action - async extraAction(button) { - await this.args.close(); - await button.action(this.args.data.post); - await this.args.data.scheduleRerender(); - } } diff --git a/app/assets/javascripts/discourse/app/components/cook-text.gjs b/app/assets/javascripts/discourse/app/components/cook-text.gjs index 51d5f11409e..426fa5e9b5b 100644 --- a/app/assets/javascripts/discourse/app/components/cook-text.gjs +++ b/app/assets/javascripts/discourse/app/components/cook-text.gjs @@ -9,19 +9,6 @@ import { resolveAllShortUrls } from "pretty-text/upload-short-url"; import { ajax } from "discourse/lib/ajax"; export default class CookText extends Component { - - {{! template-lint-disable modifier-name-case }} - - {{this.cooked}} - - - @service siteSettings; @tracked cooked = null; @@ -63,4 +50,17 @@ export default class CookText extends Component { resolveShortUrls(element) { resolveAllShortUrls(ajax, this.siteSettings, element, this.args.opts); } + + + {{! template-lint-disable modifier-name-case }} + + {{this.cooked}} + + } diff --git a/app/assets/javascripts/discourse/app/components/d-button.gjs b/app/assets/javascripts/discourse/app/components/d-button.gjs index a3480f85aef..bb7ef200eb5 100644 --- a/app/assets/javascripts/discourse/app/components/d-button.gjs +++ b/app/assets/javascripts/discourse/app/components/d-button.gjs @@ -17,62 +17,6 @@ const ACTION_AS_STRING_DEPRECATION_ARGS = [ ]; export default class DButton extends GlimmerComponentWithDeprecatedParentView { - - {{! template-lint-disable no-pointer-down-event-binding }} - - {{#if @isLoading}} - {{~icon "spinner" class="loading-icon"~}} - {{else}} - {{#if @icon}} - {{#if @ariaHidden}} - - {{~icon @icon~}} - - {{else}} - {{~icon @icon~}} - {{/if}} - {{/if}} - {{/if}} - - {{~#if this.computedLabel~}} - - {{~htmlSafe this.computedLabel~}} - {{~#if @ellipsis~}} - … - {{~/if~}} - - {{~else~}} - - {{! Zero-width space character, so icon-only button height = regular button height }} - {{~/if~}} - - {{yield}} - - - @service router; @notEmpty("args.icon") btnIcon; @@ -199,4 +143,60 @@ export default class DButton extends GlimmerComponentWithDeprecatedParentView { return false; } } + + + {{! template-lint-disable no-pointer-down-event-binding }} + + {{#if @isLoading}} + {{~icon "spinner" class="loading-icon"~}} + {{else}} + {{#if @icon}} + {{#if @ariaHidden}} + + {{~icon @icon~}} + + {{else}} + {{~icon @icon~}} + {{/if}} + {{/if}} + {{/if}} + + {{~#if this.computedLabel~}} + + {{~htmlSafe this.computedLabel~}} + {{~#if @ellipsis~}} + … + {{~/if~}} + + {{~else~}} + + {{! Zero-width space character, so icon-only button height = regular button height }} + {{~/if~}} + + {{yield}} + + } diff --git a/app/assets/javascripts/discourse/app/components/d-section.gjs b/app/assets/javascripts/discourse/app/components/d-section.gjs index 2edd783e622..cca793d4884 100644 --- a/app/assets/javascripts/discourse/app/components/d-section.gjs +++ b/app/assets/javascripts/discourse/app/components/d-section.gjs @@ -6,6 +6,18 @@ import deprecated from "discourse-common/lib/deprecated"; // Can add a body class from within a component export default class DSection extends Component { + constructor() { + super(...arguments); + deprecated( + ` is deprecated. Use {{body-class "foo-page" "bar"}} and/or instead.`, + { + since: "3.2.0.beta1", + dropFrom: "3.3.0.beta1", + id: "discourse.d-section", + } + ); + } + {{#if @pageClass}} {{bodyClass (concat @pageClass "-page")}} @@ -21,16 +33,4 @@ export default class DSection extends Component { {{yield}} {{/if}} - - constructor() { - super(...arguments); - deprecated( - ` is deprecated. Use {{body-class "foo-page" "bar"}} and/or instead.`, - { - since: "3.2.0.beta1", - dropFrom: "3.3.0.beta1", - id: "discourse.d-section", - } - ); - } } diff --git a/app/assets/javascripts/discourse/app/components/fast-edit.gjs b/app/assets/javascripts/discourse/app/components/fast-edit.gjs index faeb2e22fbc..bb3049d5d4e 100644 --- a/app/assets/javascripts/discourse/app/components/fast-edit.gjs +++ b/app/assets/javascripts/discourse/app/components/fast-edit.gjs @@ -11,29 +11,6 @@ import { on } from "@ember/modifier"; import autoFocus from "discourse/modifiers/auto-focus"; export default class FastEdit extends Component { - - {{! template-lint-disable modifier-name-case }} - {{! template-lint-disable no-pointer-down-event-binding }} - {{! template-lint-disable no-invalid-interactive }} - - {{@initialValue}} - - - - - @tracked value = this.args.initialValue; @tracked isSaving = false; @@ -81,4 +58,27 @@ export default class FastEdit extends Component { this.args.close(); } } + + + {{! template-lint-disable modifier-name-case }} + {{! template-lint-disable no-pointer-down-event-binding }} + {{! template-lint-disable no-invalid-interactive }} + + {{@initialValue}} + + + + } diff --git a/app/assets/javascripts/discourse/app/components/form-template-field/wrapper.gjs b/app/assets/javascripts/discourse/app/components/form-template-field/wrapper.gjs index 49283cb8aee..1d2e5587d52 100644 --- a/app/assets/javascripts/discourse/app/components/form-template-field/wrapper.gjs +++ b/app/assets/javascripts/discourse/app/components/form-template-field/wrapper.gjs @@ -22,28 +22,6 @@ const FormTemplateField = ; export default class FormTemplateFieldWrapper extends Component { - - {{#if this.parsedTemplate}} - - {{#each this.parsedTemplate as |content|}} - - {{/each}} - - {{else}} - - {{this.error}} - - {{/if}} - - @tracked error = null; @tracked parsedTemplate = null; @@ -94,4 +72,26 @@ export default class FormTemplateFieldWrapper extends Component { const templateContent = await response.form_template.template; return this._loadTemplate(templateContent); } + + + {{#if this.parsedTemplate}} + + {{#each this.parsedTemplate as |content|}} + + {{/each}} + + {{else}} + + {{this.error}} + + {{/if}} + } diff --git a/app/assets/javascripts/discourse/app/components/modal/dismiss-new.gjs b/app/assets/javascripts/discourse/app/components/modal/dismiss-new.gjs index e74d38b30b2..3720a4c242b 100644 --- a/app/assets/javascripts/discourse/app/components/modal/dismiss-new.gjs +++ b/app/assets/javascripts/discourse/app/components/modal/dismiss-new.gjs @@ -10,49 +10,6 @@ const REPLIES_SUBSET = "replies"; const TOPICS_SUBSET = "topics"; export default class DismissNew extends Component { - - - <:body> - - {{#if this.showDismissNewTopics}} - - {{/if}} - {{#if this.showDismissNewReplies}} - - {{/if}} - - - - <:footer> - - - - - @tracked untrack = false; @tracked dismissTopics = true; @tracked dismissPosts = true; @@ -141,4 +98,47 @@ export default class DismissNew extends Component { this.args.closeModal(); } + + + + <:body> + + {{#if this.showDismissNewTopics}} + + {{/if}} + {{#if this.showDismissNewReplies}} + + {{/if}} + + + + <:footer> + + + + } diff --git a/app/assets/javascripts/discourse/app/components/post-text-selection-toolbar.gjs b/app/assets/javascripts/discourse/app/components/post-text-selection-toolbar.gjs index 635dc72d65b..01b86156378 100644 --- a/app/assets/javascripts/discourse/app/components/post-text-selection-toolbar.gjs +++ b/app/assets/javascripts/discourse/app/components/post-text-selection-toolbar.gjs @@ -22,86 +22,6 @@ export function fixQuotes(str) { } export default class PostTextSelectionToolbar extends Component { - - {{! template-lint-disable modifier-name-case }} - {{! template-lint-disable no-invalid-interactive }} - {{! template-lint-disable no-pointer-down-event-binding }} - - - - {{#if this.embedQuoteButton}} - - {{/if}} - - {{#if @data.canEditPost}} - - {{/if}} - - {{#if this.quoteSharingEnabled}} - - {{#if this.quoteSharingShowLabel}} - - {{/if}} - - - {{#each this.quoteSharingSources as |source|}} - - {{/each}} - - - - - {{/if}} - - - - - {{#if this.isFastEditing}} - - {{/if}} - - - - - - @service currentUser; @service modal; @service site; @@ -256,4 +176,84 @@ export default class PostTextSelectionToolbar extends Component { quote: window.getSelection().toString(), }); } + + + {{! template-lint-disable modifier-name-case }} + {{! template-lint-disable no-invalid-interactive }} + {{! template-lint-disable no-pointer-down-event-binding }} + + + + {{#if this.embedQuoteButton}} + + {{/if}} + + {{#if @data.canEditPost}} + + {{/if}} + + {{#if this.quoteSharingEnabled}} + + {{#if this.quoteSharingShowLabel}} + + {{/if}} + + + {{#each this.quoteSharingSources as |source|}} + + {{/each}} + + + + + {{/if}} + + + + + {{#if this.isFastEditing}} + + {{/if}} + + + + + } diff --git a/app/assets/javascripts/discourse/app/components/post-text-selection.gjs b/app/assets/javascripts/discourse/app/components/post-text-selection.gjs index 3a36bf683bf..eb59ece255f 100644 --- a/app/assets/javascripts/discourse/app/components/post-text-selection.gjs +++ b/app/assets/javascripts/discourse/app/components/post-text-selection.gjs @@ -38,15 +38,6 @@ export function fixQuotes(str) { } export default class PostTextSelection extends Component { - - {{! template-lint-disable modifier-name-case }} - - - @service appEvents; @service capabilities; @service currentUser; @@ -284,4 +275,13 @@ export default class PostTextSelection extends Component { await this.args.selectText(); await this.hideToolbar(); } + + + {{! template-lint-disable modifier-name-case }} + + } diff --git a/app/assets/javascripts/discourse/app/components/render-glimmer-container.gjs b/app/assets/javascripts/discourse/app/components/render-glimmer-container.gjs index 353afff47cc..c1162cda512 100644 --- a/app/assets/javascripts/discourse/app/components/render-glimmer-container.gjs +++ b/app/assets/javascripts/discourse/app/components/render-glimmer-container.gjs @@ -2,6 +2,8 @@ import Component from "@glimmer/component"; import { inject as service } from "@ember/service"; export default class RenderGlimmerContainer extends Component { + @service renderGlimmer; + {{#each this.renderGlimmer._registrations as |info|}} {{#in-element info.element insertBefore=null}} @@ -12,6 +14,4 @@ export default class RenderGlimmerContainer extends Component { {{/in-element}} {{/each}} - - @service renderGlimmer; } diff --git a/app/assets/javascripts/discourse/app/components/reviewable-bundled-action.gjs b/app/assets/javascripts/discourse/app/components/reviewable-bundled-action.gjs index 21d74a3bfc7..5fedcc76821 100644 --- a/app/assets/javascripts/discourse/app/components/reviewable-bundled-action.gjs +++ b/app/assets/javascripts/discourse/app/components/reviewable-bundled-action.gjs @@ -9,6 +9,33 @@ import { action } from "@ember/object"; import { isRTL } from "discourse/lib/text-direction"; export default class ReviewableBundledAction extends Component { + @service site; + + get multiple() { + return this.args.bundle.actions.length > 1; + } + + get first() { + return this.args.bundle.actions[0]; + } + + get placement() { + const vertical = this.site.mobileView ? "top" : "bottom"; + const horizontal = isRTL() ? "end" : "start"; + + return `${vertical}-${horizontal}`; + } + + @action + perform(id) { + if (id) { + const _action = this.args.bundle.actions.find((a) => a.id === id); + this.args.performAction(_action); + } else { + this.args.performAction(this.first); + } + } + {{#if this.multiple}} {{/if}} - - @service site; - - get multiple() { - return this.args.bundle.actions.length > 1; - } - - get first() { - return this.args.bundle.actions[0]; - } - - get placement() { - const vertical = this.site.mobileView ? "top" : "bottom"; - const horizontal = isRTL() ? "end" : "start"; - - return `${vertical}-${horizontal}`; - } - - @action - perform(id) { - if (id) { - const _action = this.args.bundle.actions.find((a) => a.id === id); - this.args.performAction(_action); - } else { - this.args.performAction(this.first); - } - } } diff --git a/app/assets/javascripts/discourse/app/components/score-value.gjs b/app/assets/javascripts/discourse/app/components/score-value.gjs index 3b679b187dc..1bfb22d89be 100644 --- a/app/assets/javascripts/discourse/app/components/score-value.gjs +++ b/app/assets/javascripts/discourse/app/components/score-value.gjs @@ -2,20 +2,6 @@ import Component from "@glimmer/component"; import I18n from "I18n"; export default class ScoreValue extends Component { - - {{#if @value}} - {{if this.isNegative "-" "+"}} - - {{this.numericValue}} - {{#if @label}} - - {{this.explanationContent}} - - {{/if}} - - {{/if}} - - get numericValue() { return parseFloat(Math.abs(this.args.value)).toFixed(1); } @@ -31,4 +17,18 @@ export default class ScoreValue extends Component { get explanationContent() { return I18n.t(`review.explain.${this.args.label}.name`); } + + + {{#if @value}} + {{if this.isNegative "-" "+"}} + + {{this.numericValue}} + {{#if @label}} + + {{this.explanationContent}} + + {{/if}} + + {{/if}} + } diff --git a/app/assets/javascripts/discourse/app/components/user-tip-container.gjs b/app/assets/javascripts/discourse/app/components/user-tip-container.gjs index 8860df19b81..478e31285c9 100644 --- a/app/assets/javascripts/discourse/app/components/user-tip-container.gjs +++ b/app/assets/javascripts/discourse/app/components/user-tip-container.gjs @@ -5,6 +5,19 @@ import { action } from "@ember/object"; import { inject as service } from "@ember/service"; export default class UserTipContainer extends Component { + @service userTips; + + get safeHtmlContent() { + return htmlSafe(this.args.data.contentHtml); + } + + @action + handleDismiss(_, event) { + event.preventDefault(); + this.args.close(); + this.userTips.hideUserTipForever(this.args.data.id); + } + {{@data.titleText}} @@ -27,17 +40,4 @@ export default class UserTipContainer extends Component { {{/if}} - - @service userTips; - - get safeHtmlContent() { - return htmlSafe(this.args.data.contentHtml); - } - - @action - handleDismiss(_, event) { - event.preventDefault(); - this.args.close(); - this.userTips.hideUserTipForever(this.args.data.id); - } } diff --git a/app/assets/javascripts/discourse/app/components/user-tip.gjs b/app/assets/javascripts/discourse/app/components/user-tip.gjs index 1f8a6a0bad9..8e678d83e61 100644 --- a/app/assets/javascripts/discourse/app/components/user-tip.gjs +++ b/app/assets/javascripts/discourse/app/components/user-tip.gjs @@ -10,15 +10,6 @@ import I18n from "I18n"; import { iconHTML } from "discourse-common/lib/icon-library"; export default class UserTip extends Component { - - {{! template-lint-disable modifier-name-case }} - - {{#if this.shouldRenderTip}} - - {{/if}} - - - @service currentUser; @service userTips; @service tooltip; @@ -80,4 +71,13 @@ export default class UserTip extends Component { get shouldRenderTip() { return this.userTips.renderedId === this.args.id; } + + + {{! template-lint-disable modifier-name-case }} + + {{#if this.shouldRenderTip}} + + {{/if}} + + } diff --git a/app/assets/javascripts/float-kit/addon/components/d-float-body.gjs b/app/assets/javascripts/float-kit/addon/components/d-float-body.gjs index de7d0ee972b..cae38e28c40 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-float-body.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-float-body.gjs @@ -11,6 +11,40 @@ import TrapTab from "discourse/modifiers/trap-tab"; import DFloatPortal from "float-kit/components/d-float-portal"; export default class DFloatBody extends Component { + closeOnScroll = modifier(() => { + const firstScrollParent = getScrollParent(this.trigger); + + const handler = () => { + this.args.instance.close(); + }; + + firstScrollParent.addEventListener("scroll", handler, { passive: true }); + + return () => { + firstScrollParent.removeEventListener("scroll", handler); + }; + }); + + get supportsCloseOnClickOutside() { + return this.args.instance.expanded && this.options.closeOnClickOutside; + } + + get supportsCloseOnEscape() { + return this.args.instance.expanded && this.options.closeOnEscape; + } + + get supportsCloseOnScroll() { + return this.args.instance.expanded && this.options.closeOnScroll; + } + + get trigger() { + return this.args.instance.trigger; + } + + get options() { + return this.args.instance.options; + } + {{! template-lint-disable modifier-name-case }} - - closeOnScroll = modifier(() => { - const firstScrollParent = getScrollParent(this.trigger); - - const handler = () => { - this.args.instance.close(); - }; - - firstScrollParent.addEventListener("scroll", handler, { passive: true }); - - return () => { - firstScrollParent.removeEventListener("scroll", handler); - }; - }); - - get supportsCloseOnClickOutside() { - return this.args.instance.expanded && this.options.closeOnClickOutside; - } - - get supportsCloseOnEscape() { - return this.args.instance.expanded && this.options.closeOnEscape; - } - - get supportsCloseOnScroll() { - return this.args.instance.expanded && this.options.closeOnScroll; - } - - get trigger() { - return this.args.instance.trigger; - } - - get options() { - return this.args.instance.options; - } } diff --git a/app/assets/javascripts/float-kit/addon/components/d-float-portal.gjs b/app/assets/javascripts/float-kit/addon/components/d-float-portal.gjs index 7cc2650f825..f6f5d3a4ede 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-float-portal.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-float-portal.gjs @@ -2,6 +2,10 @@ import Component from "@glimmer/component"; import { isTesting } from "discourse-common/config/environment"; export default class DFloatPortal extends Component { + get inline() { + return this.args.inline ?? isTesting(); + } + {{#if this.inline}} {{yield}} @@ -11,8 +15,4 @@ export default class DFloatPortal extends Component { {{/in-element}} {{/if}} - - get inline() { - return this.args.inline ?? isTesting(); - } } diff --git a/app/assets/javascripts/float-kit/addon/components/d-inline-menu.gjs b/app/assets/javascripts/float-kit/addon/components/d-inline-menu.gjs index cf7e1a31266..675e49d7f01 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-inline-menu.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-inline-menu.gjs @@ -5,6 +5,8 @@ import { MENU } from "float-kit/lib/constants"; import didInsert from "@ember/render-modifiers/modifiers/did-insert"; export default class DInlineMenu extends Component { + @service menu; + {{! template-lint-disable modifier-name-case }} - - @service menu; } diff --git a/app/assets/javascripts/float-kit/addon/components/d-inline-tooltip.gjs b/app/assets/javascripts/float-kit/addon/components/d-inline-tooltip.gjs index 33958abd900..3ab13ee747e 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-inline-tooltip.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-inline-tooltip.gjs @@ -6,6 +6,8 @@ import didInsert from "@ember/render-modifiers/modifiers/did-insert"; import and from "truth-helpers/helpers/and"; export default class DInlineTooltip extends Component { + @service tooltip; + {{! template-lint-disable modifier-name-case }} - - @service tooltip; } diff --git a/app/assets/javascripts/float-kit/addon/components/d-menu.gjs b/app/assets/javascripts/float-kit/addon/components/d-menu.gjs index 362b3104847..c2edbbbb06d 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-menu.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-menu.gjs @@ -9,6 +9,49 @@ import { getOwner } from "@ember/application"; import DMenuInstance from "float-kit/lib/d-menu-instance"; export default class DMenu extends Component { + @service menu; + + @tracked menuInstance = null; + + registerTrigger = modifier((element) => { + const options = { + ...this.args, + ...{ + autoUpdate: true, + listeners: true, + beforeTrigger: () => { + this.menu.close(); + }, + }, + }; + const instance = new DMenuInstance(getOwner(this), element, options); + + this.menuInstance = instance; + + return () => { + instance.destroy(); + + if (this.isDestroying) { + this.menuInstance = null; + } + }; + }); + + get menuId() { + return `d-menu-${this.menuInstance.id}`; + } + + get options() { + return this.menuInstance?.options ?? {}; + } + + get componentArgs() { + return { + close: this.menuInstance.close, + data: this.options.data, + }; + } + {{! template-lint-disable modifier-name-case }} {{/if}} - - @service menu; - - @tracked menuInstance = null; - - registerTrigger = modifier((element) => { - const options = { - ...this.args, - ...{ - autoUpdate: true, - listeners: true, - beforeTrigger: () => { - this.menu.close(); - }, - }, - }; - const instance = new DMenuInstance(getOwner(this), element, options); - - this.menuInstance = instance; - - return () => { - instance.destroy(); - - if (this.isDestroying) { - this.menuInstance = null; - } - }; - }); - - get menuId() { - return `d-menu-${this.menuInstance.id}`; - } - - get options() { - return this.menuInstance?.options ?? {}; - } - - get componentArgs() { - return { - close: this.menuInstance.close, - data: this.options.data, - }; - } } diff --git a/app/assets/javascripts/float-kit/addon/components/d-popover.gjs b/app/assets/javascripts/float-kit/addon/components/d-popover.gjs index 508a5076454..b977390747c 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-popover.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-popover.gjs @@ -5,13 +5,6 @@ import { modifier } from "ember-modifier"; import deprecated from "discourse-common/lib/deprecated"; export default class DPopover extends Component { - - {{! template-lint-disable modifier-name-case }} - - {{yield}} - - - @service tooltip; registerDTooltip = modifier((element) => { @@ -37,4 +30,11 @@ export default class DPopover extends Component { instance.destroy(); }; }); + + + {{! template-lint-disable modifier-name-case }} + + {{yield}} + + } diff --git a/app/assets/javascripts/float-kit/addon/components/d-toasts.gjs b/app/assets/javascripts/float-kit/addon/components/d-toasts.gjs index 2f6f3149017..7935489daf6 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-toasts.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-toasts.gjs @@ -4,6 +4,8 @@ import concatClass from "discourse/helpers/concat-class"; import { on } from "@ember/modifier"; export default class DToasts extends Component { + @service toasts; + {{#each this.toasts.activeToasts as |toast|}} @@ -22,6 +24,4 @@ export default class DToasts extends Component { {{/each}} - - @service toasts; } diff --git a/app/assets/javascripts/float-kit/addon/components/d-tooltip.gjs b/app/assets/javascripts/float-kit/addon/components/d-tooltip.gjs index bcf2b6c3940..6f53520d0ba 100644 --- a/app/assets/javascripts/float-kit/addon/components/d-tooltip.gjs +++ b/app/assets/javascripts/float-kit/addon/components/d-tooltip.gjs @@ -10,6 +10,44 @@ import { getOwner } from "@ember/application"; import and from "truth-helpers/helpers/and"; export default class DTooltip extends Component { + @service tooltip; + + @tracked tooltipInstance = null; + + registerTrigger = modifier((element) => { + const options = { + ...this.args, + ...{ + listeners: true, + beforeTrigger: () => { + this.tooltip.close(); + }, + }, + }; + const instance = new DTooltipInstance(getOwner(this), element, options); + + this.tooltipInstance = instance; + + return () => { + instance.destroy(); + + if (this.isDestroying) { + this.tooltipInstance = null; + } + }; + }); + + get options() { + return this.tooltipInstance?.options; + } + + get componentArgs() { + return { + close: this.tooltip.close, + data: this.options.data, + }; + } + {{! template-lint-disable modifier-name-case }} {{/if}} - - @service tooltip; - - @tracked tooltipInstance = null; - - registerTrigger = modifier((element) => { - const options = { - ...this.args, - ...{ - listeners: true, - beforeTrigger: () => { - this.tooltip.close(); - }, - }, - }; - const instance = new DTooltipInstance(getOwner(this), element, options); - - this.tooltipInstance = instance; - - return () => { - instance.destroy(); - - if (this.isDestroying) { - this.tooltipInstance = null; - } - }; - }); - - get options() { - return this.tooltipInstance?.options; - } - - get componentArgs() { - return { - close: this.tooltip.close, - data: this.options.data, - }; - } } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-channel-info.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-channel-info.gjs index 3face999589..9ef9e5c48da 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-channel-info.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-channel-info.gjs @@ -7,6 +7,22 @@ import ChatChannelStatus from "discourse/plugins/chat/discourse/components/chat- import I18n from "I18n"; export default class ChatChannelMessageEmojiPicker extends Component { + @service chatChannelInfoRouteOriginManager; + @service site; + + membersLabel = I18n.t("chat.channel_info.tabs.members"); + settingsLabel = I18n.t("chat.channel_info.tabs.settings"); + backToChannelLabel = I18n.t("chat.channel_info.back_to_all_channel"); + backToAllChannelsLabel = I18n.t("chat.channel_info.back_to_channel"); + + get showTabs() { + return ( + this.site.desktopView && + this.args.channel.membershipsCount > 1 && + this.args.channel.isOpen + ); + } + @@ -66,20 +82,4 @@ export default class ChatChannelMessageEmojiPicker extends Component { {{outlet}} - - @service chatChannelInfoRouteOriginManager; - @service site; - - membersLabel = I18n.t("chat.channel_info.tabs.members"); - settingsLabel = I18n.t("chat.channel_info.tabs.settings"); - backToChannelLabel = I18n.t("chat.channel_info.back_to_all_channel"); - backToAllChannelsLabel = I18n.t("chat.channel_info.back_to_channel"); - - get showTabs() { - return ( - this.site.desktopView && - this.args.channel.membershipsCount > 1 && - this.args.channel.isOpen - ); - } } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-channel-members.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-channel-members.gjs index bc102c97d83..a1fc0646597 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-channel-members.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-channel-members.gjs @@ -14,45 +14,6 @@ import { hash } from "@ember/helper"; import { schedule } from "@ember/runloop"; export default class ChatChannelMembers extends Component { - - {{! template-lint-disable modifier-name-case }} - - - - {{#if (gt @channel.membershipsCount 0)}} - - {{#each this.members as |membership|}} - - - - {{else}} - {{#if this.noResults}} - - {{this.noMembershipsFoundLabel}} - - {{/if}} - {{/each}} - - - - - - {{else}} - - {{this.noMembershipsLabel}} - - {{/if}} - - - @service chatApi; @service modal; @service loadingSlider; @@ -122,4 +83,43 @@ export default class ChatChannelMembers extends Component { await this.members.load({ limit: 20 }); this.loadingSlider.transitionEnded(); } + + + {{! template-lint-disable modifier-name-case }} + + + + {{#if (gt @channel.membershipsCount 0)}} + + {{#each this.members as |membership|}} + + + + {{else}} + {{#if this.noResults}} + + {{this.noMembershipsFoundLabel}} + + {{/if}} + {{/each}} + + + + + + {{else}} + + {{this.noMembershipsLabel}} + + {{/if}} + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-channel-message-emoji-picker.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-channel-message-emoji-picker.gjs index 0861363f1d2..51e4ed1f24d 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-channel-message-emoji-picker.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-channel-message-emoji-picker.gjs @@ -7,18 +7,6 @@ import ChatEmojiPicker from "discourse/plugins/chat/discourse/components/chat-em import { modifier } from "ember-modifier"; export default class ChatChannelMessageEmojiPicker extends Component { - - {{! template-lint-disable modifier-name-case }} - - - @service site; @service chatEmojiPickerManager; @@ -74,4 +62,16 @@ export default class ChatChannelMessageEmojiPicker extends Component { willDestroy() { this._popper?.destroy(); } + + + {{! template-lint-disable modifier-name-case }} + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-channel-row.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-channel-row.gjs index 98eace72938..50904d387f3 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-channel-row.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-channel-row.gjs @@ -22,70 +22,6 @@ import { htmlSafe } from "@ember/template"; const FADEOUT_CLASS = "-fade-out"; export default class ChatChannelRow extends Component { - - {{! template-lint-disable modifier-name-case }} - - - - - - {{#if - (and @options.leaveButton @channel.isFollowing this.site.desktopView) - }} - - {{/if}} - - - {{#if this.showRemoveButton}} - - {{icon "times-circle"}} - - {{/if}} - - - @service api; @service capabilities; @service chat; @@ -219,4 +155,68 @@ export default class ChatChannelRow extends Component { stopTrackingStatus() { this.#firstDirectMessageUser?.stopTrackingStatus(); } + + + {{! template-lint-disable modifier-name-case }} + + + + + + {{#if + (and @options.leaveButton @channel.isFollowing this.site.desktopView) + }} + + {{/if}} + + + {{#if this.showRemoveButton}} + + {{icon "times-circle"}} + + {{/if}} + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-channel-settings.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-channel-settings.gjs index 7439051feb8..f80994ed346 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-channel-settings.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-channel-settings.gjs @@ -27,279 +27,6 @@ const NOTIFICATION_LEVELS = [ ]; export default class ChatAboutScreen extends Component { - - - - {{#if this.shouldRenderTitleSection}} - - - <:default> - - {{replaceEmoji @channel.title}} - - - {{#if @channel.isCategoryChannel}} - - - /chat/c/{{@channel.slug}}/{{@channel.id}} - - - {{/if}} - - - <:action> - {{#if this.canEditChannel}} - - {{/if}} - - - - - {{/if}} - - {{#if this.shouldRenderDescriptionSection}} - - - <:default> - {{#if @channel.description.length}} - {{@channel.description}} - {{else}} - {{this.descriptionPlaceholder}} - {{/if}} - - - <:action> - {{#if this.canEditChannel}} - - {{/if}} - - - - {{/if}} - - {{#if this.site.mobileView}} - - - - {{/if}} - - {{#if @channel.isOpen}} - - - <:action> - - - - - {{#if this.shouldRenderDesktopNotificationsLevelSection}} - - <:action> - - - - {{/if}} - - {{#if this.shouldRenderMobileNotificationsLevelSection}} - - <:action> - - - - {{/if}} - - {{/if}} - - - {{#if @channel.isCategoryChannel}} - - {{categoryBadge - @channel.chatable - link=true - allowUncategorized=true - }} - - {{/if}} - - - - - - - {{#if this.shouldRenderAdminSection}} - - {{#if this.autoJoinAvailable}} - - <:action> - - - - {{/if}} - - {{#if this.toggleChannelWideMentionsAvailable}} - - <:action> - - - - <:description> - {{this.channelWideMentionsDescription}} - - - {{/if}} - - {{#if this.toggleThreadingAvailable}} - - <:action> - - - - <:description> - {{this.toggleThreadingDescription}} - - - {{/if}} - - {{#if this.shouldRenderStatusSection}} - {{#if this.shouldRenderArchiveRow}} - - <:action> - - - - {{/if}} - - - <:action> - {{#if @channel.isOpen}} - - {{else}} - - {{/if}} - - - - - <:action> - - - - {{/if}} - - - {{/if}} - - - - <:action> - - - - - - - - @service chatApi; @service chatGuardian; @service currentUser; @@ -578,4 +305,277 @@ export default class ChatAboutScreen extends Component { model: this.args.channel, }); } + + + + + {{#if this.shouldRenderTitleSection}} + + + <:default> + + {{replaceEmoji @channel.title}} + + + {{#if @channel.isCategoryChannel}} + + + /chat/c/{{@channel.slug}}/{{@channel.id}} + + + {{/if}} + + + <:action> + {{#if this.canEditChannel}} + + {{/if}} + + + + + {{/if}} + + {{#if this.shouldRenderDescriptionSection}} + + + <:default> + {{#if @channel.description.length}} + {{@channel.description}} + {{else}} + {{this.descriptionPlaceholder}} + {{/if}} + + + <:action> + {{#if this.canEditChannel}} + + {{/if}} + + + + {{/if}} + + {{#if this.site.mobileView}} + + + + {{/if}} + + {{#if @channel.isOpen}} + + + <:action> + + + + + {{#if this.shouldRenderDesktopNotificationsLevelSection}} + + <:action> + + + + {{/if}} + + {{#if this.shouldRenderMobileNotificationsLevelSection}} + + <:action> + + + + {{/if}} + + {{/if}} + + + {{#if @channel.isCategoryChannel}} + + {{categoryBadge + @channel.chatable + link=true + allowUncategorized=true + }} + + {{/if}} + + + + + + + {{#if this.shouldRenderAdminSection}} + + {{#if this.autoJoinAvailable}} + + <:action> + + + + {{/if}} + + {{#if this.toggleChannelWideMentionsAvailable}} + + <:action> + + + + <:description> + {{this.channelWideMentionsDescription}} + + + {{/if}} + + {{#if this.toggleThreadingAvailable}} + + <:action> + + + + <:description> + {{this.toggleThreadingDescription}} + + + {{/if}} + + {{#if this.shouldRenderStatusSection}} + {{#if this.shouldRenderArchiveRow}} + + <:action> + + + + {{/if}} + + + <:action> + {{#if @channel.isOpen}} + + {{else}} + + {{/if}} + + + + + <:action> + + + + {{/if}} + + + {{/if}} + + + + <:action> + + + + + + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-desktop.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-desktop.gjs index 645070f6bf6..c7aeedd788b 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-desktop.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-desktop.gjs @@ -25,6 +25,91 @@ const REDUCED = "reduced"; const REDUCED_WIDTH_THRESHOLD = 500; export default class ChatMessageActionsDesktop extends Component { + @service chat; + @service chatEmojiPickerManager; + @service site; + + @tracked size = FULL; + + popper = null; + + get message() { + return this.chat.activeMessage.model; + } + + get context() { + return this.chat.activeMessage.context; + } + + get messageInteractor() { + return new ChatMessageInteractor( + getOwner(this), + this.message, + this.context + ); + } + + get shouldRenderFavoriteReactions() { + return this.size === FULL; + } + + @action + onWheel() { + // prevents menu to stop scroll on the list of messages + this.chat.activeMessage = null; + } + + @action + setup(element) { + this.popper?.destroy(); + + schedule("afterRender", () => { + const messageContainer = chatMessageContainer( + this.message.id, + this.context + ); + + if (!messageContainer) { + return; + } + + const viewport = messageContainer.closest(".popper-viewport"); + this.size = + viewport.clientWidth < REDUCED_WIDTH_THRESHOLD ? REDUCED : FULL; + + if (!messageContainer) { + return; + } + + this.popper = createPopper(messageContainer, element, { + placement: "top-end", + strategy: "fixed", + modifiers: [ + { + name: "flip", + enabled: true, + options: { + boundary: viewport, + fallbackPlacements: ["bottom-end"], + }, + }, + { name: "hide", enabled: true }, + { name: "eventListeners", options: { scroll: false } }, + { + name: "offset", + options: { offset: [-2, MSG_ACTIONS_VERTICAL_PADDING] }, + }, + ], + }); + }); + } + + @action + teardown() { + this.popper?.destroy(); + this.popper = null; + } + {{! template-lint-disable modifier-name-case }} {{#if (and this.site.desktopView this.chat.activeMessage.model.persisted)}} @@ -112,89 +197,4 @@ export default class ChatMessageActionsDesktop extends Component { {{/if}} - - @service chat; - @service chatEmojiPickerManager; - @service site; - - @tracked size = FULL; - - popper = null; - - get message() { - return this.chat.activeMessage.model; - } - - get context() { - return this.chat.activeMessage.context; - } - - get messageInteractor() { - return new ChatMessageInteractor( - getOwner(this), - this.message, - this.context - ); - } - - get shouldRenderFavoriteReactions() { - return this.size === FULL; - } - - @action - onWheel() { - // prevents menu to stop scroll on the list of messages - this.chat.activeMessage = null; - } - - @action - setup(element) { - this.popper?.destroy(); - - schedule("afterRender", () => { - const messageContainer = chatMessageContainer( - this.message.id, - this.context - ); - - if (!messageContainer) { - return; - } - - const viewport = messageContainer.closest(".popper-viewport"); - this.size = - viewport.clientWidth < REDUCED_WIDTH_THRESHOLD ? REDUCED : FULL; - - if (!messageContainer) { - return; - } - - this.popper = createPopper(messageContainer, element, { - placement: "top-end", - strategy: "fixed", - modifiers: [ - { - name: "flip", - enabled: true, - options: { - boundary: viewport, - fallbackPlacements: ["bottom-end"], - }, - }, - { name: "hide", enabled: true }, - { name: "eventListeners", options: { scroll: false } }, - { - name: "offset", - options: { offset: [-2, MSG_ACTIONS_VERTICAL_PADDING] }, - }, - ], - }); - }); - } - - @action - teardown() { - this.popper?.destroy(); - this.popper = null; - } } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-mobile.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-mobile.gjs index 52c34c1f54c..0dc7e68bd3f 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-mobile.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-message-actions-mobile.gjs @@ -17,6 +17,86 @@ import or from "truth-helpers/helpers/or"; import BookmarkIcon from "discourse/components/bookmark-icon"; export default class ChatMessageActionsMobile extends Component { + @service chat; + @service site; + @service capabilities; + + @tracked hasExpandedReply = false; + @tracked showFadeIn = false; + + get message() { + return this.chat.activeMessage.model; + } + + get context() { + return this.chat.activeMessage.context; + } + + get messageInteractor() { + return new ChatMessageInteractor( + getOwner(this), + this.message, + this.context + ); + } + + @action + fadeAndVibrate() { + discourseLater(this.#addFadeIn.bind(this)); + + if (this.capabilities.userHasBeenActive && this.capabilities.canVibrate) { + navigator.vibrate(5); + } + } + + @action + expandReply(event) { + event.stopPropagation(); + this.hasExpandedReply = true; + } + + @action + collapseMenu(event) { + event.preventDefault(); + this.#onCloseMenu(); + } + + @action + actAndCloseMenu(fnId) { + this.messageInteractor[fnId](); + this.#onCloseMenu(); + } + + @action + openEmojiPicker(_, event) { + this.messageInteractor.openEmojiPicker(_, event); + this.#onCloseMenu(); + } + + #onCloseMenu() { + this.#removeFadeIn(); + + // we don't want to remove the component right away as it's animating + // 200 is equal to the duration of the css animation + discourseLater(() => { + if (this.isDestroying || this.isDestroyed) { + return; + } + + // by ensuring we are not hovering any message anymore + // we also ensure the menu is fully removed + this.chat.activeMessage = null; + }, 200); + } + + #addFadeIn() { + this.showFadeIn = true; + } + + #removeFadeIn() { + this.showFadeIn = false; + } + {{! template-lint-disable modifier-name-case }} {{#if (and this.site.mobileView this.chat.activeMessage.model.persisted)}} @@ -113,84 +193,4 @@ export default class ChatMessageActionsMobile extends Component { {{/if}} - - @service chat; - @service site; - @service capabilities; - - @tracked hasExpandedReply = false; - @tracked showFadeIn = false; - - get message() { - return this.chat.activeMessage.model; - } - - get context() { - return this.chat.activeMessage.context; - } - - get messageInteractor() { - return new ChatMessageInteractor( - getOwner(this), - this.message, - this.context - ); - } - - @action - fadeAndVibrate() { - discourseLater(this.#addFadeIn.bind(this)); - - if (this.capabilities.userHasBeenActive && this.capabilities.canVibrate) { - navigator.vibrate(5); - } - } - - @action - expandReply(event) { - event.stopPropagation(); - this.hasExpandedReply = true; - } - - @action - collapseMenu(event) { - event.preventDefault(); - this.#onCloseMenu(); - } - - @action - actAndCloseMenu(fnId) { - this.messageInteractor[fnId](); - this.#onCloseMenu(); - } - - @action - openEmojiPicker(_, event) { - this.messageInteractor.openEmojiPicker(_, event); - this.#onCloseMenu(); - } - - #onCloseMenu() { - this.#removeFadeIn(); - - // we don't want to remove the component right away as it's animating - // 200 is equal to the duration of the css animation - discourseLater(() => { - if (this.isDestroying || this.isDestroyed) { - return; - } - - // by ensuring we are not hovering any message anymore - // we also ensure the menu is fully removed - this.chat.activeMessage = null; - }, 200); - } - - #addFadeIn() { - this.showFadeIn = true; - } - - #removeFadeIn() { - this.showFadeIn = false; - } } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-message-reaction.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-message-reaction.gjs index e90f524c3a2..9d715ee0b90 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-message-reaction.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-message-reaction.gjs @@ -11,38 +11,6 @@ import and from "truth-helpers/helpers/and"; import concatClass from "discourse/helpers/concat-class"; export default class ChatMessageReaction extends Component { - - {{! template-lint-disable modifier-name-case }} - {{#if (and @reaction this.emojiUrl)}} - - - - {{#if (and this.showCount @reaction.count)}} - {{@reaction.count}} - {{/if}} - - {{/if}} - - @service capabilities; @service currentUser; @service tooltip; @@ -101,4 +69,36 @@ export default class ChatMessageReaction extends Component { return emojiUnescape(getReactionText(this.args.reaction, this.currentUser)); } + + + {{! template-lint-disable modifier-name-case }} + {{#if (and @reaction this.emojiUrl)}} + + + + {{#if (and this.showCount @reaction.count)}} + {{@reaction.count}} + {{/if}} + + {{/if}} + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-message.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-message.gjs index f8ef8cfb2b1..aa05cf69c6b 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-message.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-message.gjs @@ -48,140 +48,6 @@ export const MENTION_KEYWORDS = ["here", "all"]; export const MESSAGE_CONTEXT_THREAD = "thread"; export default class ChatMessage extends Component { - - {{! template-lint-disable no-invalid-interactive }} - {{! template-lint-disable modifier-name-case }} - {{#if this.shouldRender}} - {{#if (eq @context "channel")}} - - - {{/if}} - - - {{#if this.show}} - {{#if this.pane.selectingMessages}} - - {{/if}} - - {{#if this.deletedAndCollapsed}} - - - - {{else if this.hiddenAndCollapsed}} - - - - {{else}} - - {{#unless this.hideReplyToInfo}} - - {{/unless}} - - {{#if this.hideUserInfo}} - - {{else}} - - {{/if}} - - - - - - {{#if @message.reactions.length}} - - {{#each @message.reactions as |reaction|}} - - {{/each}} - - {{#if this.shouldRenderOpenEmojiPickerButton}} - - {{/if}} - - {{/if}} - - - - - - {{#if this.showThreadIndicator}} - - {{/if}} - - {{/if}} - {{/if}} - - {{/if}} - - @service site; @service dialog; @service currentUser; @@ -624,4 +490,138 @@ export default class ChatMessage extends Component { user.off("status-changed", this, "refreshStatusOnMentions"); }); } + + + {{! template-lint-disable no-invalid-interactive }} + {{! template-lint-disable modifier-name-case }} + {{#if this.shouldRender}} + {{#if (eq @context "channel")}} + + + {{/if}} + + + {{#if this.show}} + {{#if this.pane.selectingMessages}} + + {{/if}} + + {{#if this.deletedAndCollapsed}} + + + + {{else if this.hiddenAndCollapsed}} + + + + {{else}} + + {{#unless this.hideReplyToInfo}} + + {{/unless}} + + {{#if this.hideUserInfo}} + + {{else}} + + {{/if}} + + + + + + {{#if @message.reactions.length}} + + {{#each @message.reactions as |reaction|}} + + {{/each}} + + {{#if this.shouldRenderOpenEmojiPickerButton}} + + {{/if}} + + {{/if}} + + + + + + {{#if this.showThreadIndicator}} + + {{/if}} + + {{/if}} + {{/if}} + + {{/if}} + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-retention-reminder-text.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-retention-reminder-text.gjs index 71fc34beca8..6915cca0ce7 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-retention-reminder-text.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-retention-reminder-text.gjs @@ -3,12 +3,6 @@ import I18n from "I18n"; import { inject as service } from "@ember/service"; export default class ChatRetentionReminderText extends Component { - - - {{this.text}} - - - @service siteSettings; get text() { @@ -36,4 +30,10 @@ export default class ChatRetentionReminderText extends Component { ? this.siteSettings.chat_dm_retention_days : this.siteSettings.chat_channel_retention_days; } + + + + {{this.text}} + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat-user-avatar.gjs b/plugins/chat/assets/javascripts/discourse/components/chat-user-avatar.gjs index d1907d23b42..7c250654a6f 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat-user-avatar.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat-user-avatar.gjs @@ -5,25 +5,6 @@ import { htmlSafe } from "@ember/template"; import concatClass from "discourse/helpers/concat-class"; export default class ChatUserAvatar extends Component { - - - {{#if this.interactive}} - - {{this.avatar}} - - {{else}} - {{this.avatar}} - {{/if}} - - - @service chat; get avatar() { @@ -55,4 +36,23 @@ export default class ChatUserAvatar extends Component { ) ); } + + + + {{#if this.interactive}} + + {{this.avatar}} + + {{else}} + {{this.avatar}} + {{/if}} + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat/form/section.gjs b/plugins/chat/assets/javascripts/discourse/components/chat/form/section.gjs index 47dcf2e8530..d12bb2a6004 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat/form/section.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat/form/section.gjs @@ -2,6 +2,10 @@ import Component from "@glimmer/component"; import ChatFormRow from "discourse/plugins/chat/discourse/components/chat/form/row"; export default class ChatFormSection extends Component { + get yieldableArgs() { + return { row: ChatFormRow }; + } + {{#if @title}} @@ -15,8 +19,4 @@ export default class ChatFormSection extends Component { - - get yieldableArgs() { - return { row: ChatFormRow }; - } } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat/header/icon.gjs b/plugins/chat/assets/javascripts/discourse/components/chat/header/icon.gjs index 6e4ae63b1f7..7abe016b443 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat/header/icon.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat/header/icon.gjs @@ -8,24 +8,6 @@ import concatClass from "discourse/helpers/concat-class"; import I18n from "I18n"; export default class ChatHeaderIcon extends Component { - - - {{~icon this.icon~}} - {{#if this.showUnreadIndicator}} - - {{/if}} - - - @service currentUser; @service site; @service chatStateManager; @@ -96,4 +78,22 @@ export default class ChatHeaderIcon extends Component { return getURL(this.chatStateManager.lastKnownChatURL || "/chat"); } + + + + {{~icon this.icon~}} + {{#if this.showUnreadIndicator}} + + {{/if}} + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat/header/icon/unread-indicator.gjs b/plugins/chat/assets/javascripts/discourse/components/chat/header/icon/unread-indicator.gjs index 0932f3cdd4d..70628ed946b 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat/header/icon/unread-indicator.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat/header/icon/unread-indicator.gjs @@ -9,18 +9,6 @@ import { const MAX_UNREAD_COUNT = 99; export default class ChatHeaderIconUnreadIndicator extends Component { - - {{#if this.showUrgentIndicator}} - - - {{this.unreadCountLabel}} - - - {{else if this.showUnreadIndicator}} - - {{/if}} - - @service chatTrackingStateManager; @service currentUser; @@ -78,4 +66,16 @@ export default class ChatHeaderIconUnreadIndicator extends Component { return preferences.includes(this.indicatorPreference); } + + + {{#if this.showUrgentIndicator}} + + + {{this.unreadCountLabel}} + + + {{else if this.showUnreadIndicator}} + + {{/if}} + } diff --git a/plugins/chat/assets/javascripts/discourse/components/chat/selection-manager.gjs b/plugins/chat/assets/javascripts/discourse/components/chat/selection-manager.gjs index 610dd1dfa2a..a56315b4844 100644 --- a/plugins/chat/assets/javascripts/discourse/components/chat/selection-manager.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/chat/selection-manager.gjs @@ -12,49 +12,6 @@ import not from "truth-helpers/helpers/not"; import I18n from "I18n"; export default class ChatSelectionManager extends Component { - - - - - - - - {{#if this.enableMove}} - - {{/if}} - - - - - - @service("composer") topicComposer; @service router; @service modal; @@ -150,4 +107,47 @@ export default class ChatSelectionManager extends Component { popupAjaxError(error); } } + + + + + + + + + {{#if this.enableMove}} + + {{/if}} + + + + + } diff --git a/plugins/chat/assets/javascripts/discourse/components/dc-filter-input.gjs b/plugins/chat/assets/javascripts/discourse/components/dc-filter-input.gjs index 5d924d93822..0c401f0d5e6 100644 --- a/plugins/chat/assets/javascripts/discourse/components/dc-filter-input.gjs +++ b/plugins/chat/assets/javascripts/discourse/components/dc-filter-input.gjs @@ -8,6 +8,25 @@ import { modifier } from "ember-modifier"; import { tracked } from "@glimmer/tracking"; export default class DcFilterInput extends Component { + @tracked isFocused = false; + + focusState = modifier((element) => { + const focusInHandler = () => { + this.isFocused = true; + }; + const focusOutHandler = () => { + this.isFocused = false; + }; + + element.addEventListener("focusin", focusInHandler); + element.addEventListener("focusout", focusOutHandler); + + return () => { + element.removeEventListener("focusin", focusInHandler); + element.removeEventListener("focusout", focusOutHandler); + }; + }); + {{! template-lint-disable modifier-name-case }} - - @tracked isFocused = false; - - focusState = modifier((element) => { - const focusInHandler = () => { - this.isFocused = true; - }; - const focusOutHandler = () => { - this.isFocused = false; - }; - - element.addEventListener("focusin", focusInHandler); - element.addEventListener("focusout", focusOutHandler); - - return () => { - element.removeEventListener("focusin", focusInHandler); - element.removeEventListener("focusout", focusOutHandler); - }; - }); }
- {{#if this.showDismissNewTopics}} - - {{/if}} - {{#if this.showDismissNewReplies}} - - {{/if}} - -
+ {{#if this.showDismissNewTopics}} + + {{/if}} + {{#if this.showDismissNewReplies}} + + {{/if}} + +
- {{this.noMembershipsLabel}} -
+ {{this.noMembershipsLabel}} +