Opened 10 years ago

Closed 7 years ago

Last modified 7 years ago

#723 closed defect (bug) (fixed)

No link to log in to Make sites on mobile devices

Reported by: iandunn's profile iandunn Owned by: pento's profile pento
Milestone: Priority: normal
Component: Make (Get Involved) / P2 Keywords:


On small screens the Admin Bar is hidden, so there's no link to log in.

Attachments (2)

logged-out-admin-bar.rtl.patch (467 bytes) - added by SergeyBiryukov 7 years ago.
logged-out-admin-bar.rtl.2.patch (392 bytes) - added by SergeyBiryukov 7 years ago.

Download all attachments as: .zip

Change History (12)

This ticket was mentioned in Slack in #meta by iandunn. View the logs.

10 years ago

#2 @melchoyce
9 years ago

  • Keywords needs-ui needs-patch added

#3 @samuelsidler
7 years ago

  • Owner set to pento
  • Status changed from new to assigned

#5 @SergeyBiryukov
7 years ago

Added in [4684] as a fix for #2376.

#6 follow-up: @pento
7 years ago

  • Keywords needs-ui needs-patch removed
  • Resolution set to fixed
  • Status changed from assigned to closed

#7 in reply to: ↑ 6 @SergeyBiryukov
7 years ago

Replying to pento:

I put it in the plugin, instead.

I wanted to revert [4684] now that it's redundant, but the plugin needs a tweak for RTL.

Any chance to commit logged-out-admin-bar.rtl.patch as well?

#8 @SergeyBiryukov
7 years ago

On second thought, we could adjust the padding to work for both LTR and RTL: logged-out-admin-bar.rtl.2.patch.

#9 @pento
7 years ago

Done, thanks @SergeyBiryukov!

#10 @SergeyBiryukov
7 years ago

In 4858:

Support Theme: Revert [4684], as it's now redundant.

CSS for login/register buttons on mobile devices was added to the "Logged Out Admin Bar" plugin and now works for both support forums and sites.

See #723, #2376.

Note: See TracTickets for help on using tickets.