Padlock icons in room header

[foldleft]

Fixes element-hq/element-web#11832

Encrypted, locked:
Unencrypted, locked:
Public room:

[jryans]

@foldleft, since this is a UI change, it would be great to include a screenshot in the PR body. Then we can also get @nadonomy to review it from a design perspective.

[foldleft]

@foldleft, since this is a UI change, it would be great to include a screenshot in the PR body. Then we can also get @nadonomy to review it from a design perspective.

oops, I totally took some

[jryans]

oops, I totally took some

Great, I added @nadonomy as an extra reviewer for the design side.

[t3chguy]

Code-wise LGTM, deferring to Nad for design review

[jryans]

According to our new process experiment discussed yesterday, now that it's guarded by a labs flag, we have the option of skipping design review on the PR. (I'll leave it to @foldleft to decide whether to wait for it or not.)

[nadonomy]

Is it guarded by the same labs flag as the general cross-signing one? If so then I don't see a problem with merging.

When we move cross-signing from /develop to production we'll be doing rounds of small visual polish and there'll be opportunity to action the required polish then.

[jryans]

Is it guarded by the same labs flag as the general cross-signing one? If so then I don't see a problem with merging.

Yes, this and anything cross-signing related are all using a single labs flag. (The only outlier is the shield in the room header which already shipped for all in December, but going forward at least, all should be guarded.)