Show Original Poster Avatar supresses Bulk Actions

Lhc_fl · June 11, 2023, 4:23pm

BTW: The code for this theme component is outdated and doesn’t contain the following code, which prevents bulk-select functionality for admins on mobile.

github.com

Lhcfl/discourse-mobile-op-avatar-component/blob/894d6ed94b929c66eca58a4ab004ca5eb9b33515/javascripts/discourse/templates/mobile/list/topic-list-item.hbr#LL4C1-L11C12


      
          {{#if bulkSelectEnabled}}
            <label for="bulk-select-{{topic.id}}">
              <input type="checkbox" class="bulk-select" id="bulk-select-{{topic.id}}">
            </label>
          {{else}}
            {{!-- updated to show OP avatar --}}
            <a href="{{topic.firstPostUrl}}"  data-user-card="{{topic.creator.username}}">{{avatar topic.creator imageSize="large"}}</a>
          {{/if}}

I opened a PR to fix this, hope you will review it

awesomerobot · June 13, 2023, 4:17pm

Thanks @Lhc_fl! I’ve merged your PR.

Topic		Replies	Views
Show Original Poster Avatars Theme component mobile , official , original-poster-avatar	2	4119	June 26, 2024
On the mobile theme, the list of posts show's the latest replying user's avatar, not the OP's avatar UX	14	3676	November 25, 2020
Extend Show Original Poster Avatar to also work on desktop UX original-poster-avatar	5	20	September 22, 2023
Avatar shows over banner on mobile Bug	3	1308	March 24, 2015
Display Topic Author on Topic List UX	7	2422	February 8, 2021

Show Original Poster Avatar supresses Bulk Actions

Related topics