A minor UX bug in discourse-subscriptions.
This PR solved the blank space occuring when visiting other users’ profile page.
main ← small-lovely-cat:discourse-subscriptions-whitespace-fix
merged 10:30AM - 03 Dec 25 UTC
The original discourse-subscriptions plugin uses a condition rendering logic to … make sure the button doesn't exist on other users' main page.
However, the logic doesn't consider the parent node `<li id="ember136" class="user-main-nav-outlet billing ember-view">`. As a matter of fact, there exists a blank that takes space and make the nav-bar misaligned.
This PR change the condition rendering logic `{{#if this.viewingSelf}}` to the class `@classNameBindings("viewingSelf::hidden")`, so that the whole element can hide entirely without leaving a blank space.
Before:
<img width="918" height="750" alt="image" src="https://github.com/user-attachments/assets/79cf0886-a77c-467c-a00c-b06a3ae3a2eb" />
After:
<img width="982" height="837" alt="image" src="https://github.com/user-attachments/assets/cea5139c-ffff-4a99-a1fb-b537ebc98c94" />
ReFix the loading loop when a topic is removed.
main ← small-lovely-cat:post-stream-loading-loop-fix
merged 07:28PM - 09 Dec 25 UTC
In #35409, a fix has been implemented to deal with the loading loop when a topic… is removed.
However, that PR hasn't fully solved that problem, when admin enable the new post-stream setting, the fetch api will be `findPostsByIds`->`loadIntoIdentityMap` rather than the original `fetchNextWindow`, causing a regression.
This PR adds the same logic to the `loadIntoIdentityMap` function, so that the logic work as well in the new setting.
This PR adds the same logic to the loadIntoIdentityMap api that redeals with the loading loop when a post is removed, when the new post-stream is enabled.
Please take time to have them checked.
@cvx @pmusaraj
4 לייקים
There’s still a PR not dealt with, please have a look when free.
Thanks~
@zogstrip
לייק 1
saquetim
9 בדצמבר, 2025, 7:29pm
3
I just merged the PR @small-lovely-cat .
Thank you for catching this one.
2 לייקים