github-mirror/discourse
github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-04-24 22:14:31 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

DEV: remove empty outlet wrapper to prevent it from occupying layout space (#31391)

Browse Source 
The `toggle-summary` class is specific to our use in the AI plugin, so
I've moved it there in
https://github.com/discourse/discourse-ai/pull/1136...

This wrapper was occupying some space even though it was empty in the PM
topic map, so it's better to not have it.

This also allows us to remove some sub-optimal CSS that was adding a gap
based on content within it. Without the empty wrapper, we can always
have the gap present.
This commit is contained in:
Kris 2025-02-19 12:58:21 -05:00 committed by GitHub
parent f0d231355e
commit c1ee4e120e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
app/assets/javascripts/discourse/app/components/topic-map/index.gjs
View File

@ -17,13 +17,11 @@ const TopicMap = <template>
/>
</section>
<section class="topic-map__additional-contents toggle-summary">
<PluginOutlet
@name="topic-map-expanded-after"
@defaultGlimmer={{true}}
@outletArgs={{hash topic=@model postStream=@postStream}}
/>
</section>
<PluginOutlet
@name="topic-map-expanded-after"
@defaultGlimmer={{true}}
@outletArgs={{hash topic=@model postStream=@postStream}}
/>
{{#if @showPMMap}}
<section class="topic-map__private-message-map">

5
app/assets/stylesheets/common/components/topic-map.scss
View File

@ -24,10 +24,7 @@ body:not(.archetype-private_message) {
align-items: center;
justify-content: space-between;
flex-wrap: wrap;
&:has(.topic-map__additional-contents *) {
column-gap: 0.75em;
}
column-gap: 0.75em;
}
}