654677 - Hide "be awesome" sidebar when logged in

Status: VERIFIED FIXED

(support.mozilla.org :: Knowledge Base Software, task)

Description

[Kadir Topal [:atopal]]

The second sidebar is not needed for people who are already contributors and makes it impossible to resize the text area. Please remove the second sidebar for contributors.

Comment 1

[Kadir Topal [:atopal]]

Attachment: what I mean with 2nd sidebar

Comment 2

[Ricky Rosario [:rrosario, :r1cky]]

https://github.com/jsocol/kitsune/commit/224fc85103d2f941cd75ea9cf290d34c753a2cc0

Comment 3

[Kadir Topal [:atopal]]

Thanks for the fix Ricky. Just to make sure, since I was not explicit, this only removes the sidebar on pages with a text area (/edit), right?

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to comment #3)
> Thanks for the fix Ricky. Just to make sure, since I was not explicit, this
> only removes the sidebar on pages with a text area (/edit), right?

This change removes it across the board. So they will only see the "For Contributors" box. I think it looked ok, but if you want something else then we can workout which pages to hide them instead.

Comment 5

[Kadir Topal [:atopal]]

Yes, we still want to show that sidebar to people who have for example logged-in to get help in the forums. As far as I can tell, right now it only needs to be excluded on /translate

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

OK, that should be easy.

Comment 7

[Ricky Rosario [:rrosario, :r1cky]]

Only hiding on translate page now:
https://github.com/jsocol/kitsune/commit/c03231430048c0dccfaff38d5d52377ea0c13105

Comment 8

[Kadir Topal [:atopal]]

Perfect, thanks!

Comment 9

[Rebecca Billings [:rbillings]]

Verified kb/article/translate does not display the 'be awesome' sidebar module