WordPress.org
Get WordPress

Making WordPress.org

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#2213 closed defect (bug) (fixed)

Coming Soon page shows broken admin bar for logged in users

Reported by: ryelle's profile ryelle Owned by: iandunn's profile iandunn
Milestone: Priority: normal
Component: WordCamp Site & Plugins Keywords: commit
Cc:

Description

The coming soon page was designed for logged out users, so the admin bar was not taken into account - a "logged in user" was assumed to be the site organizer (who passes through into the site). Users might be logged into the wordcamp network, but not site admins, so they should see both the coming soon page and the admin bar.

The CSS is not loaded for the admin bar, thus it just dumps out links in the page's footer. See attachment for broken-looking page.

Patch incoming to re-add the CSS for logged in users.

Attachments (2)

FireShot Capture 003 - WordCamp Torino - https___2017.torino.wordcamp.org_#wp-toolbar.jpg​ (192.8 KB) - added by ryelle 10 years ago.
coming-soon-ab.diff​ (638 bytes) - added by ryelle 10 years ago.

Download all attachments as: .zip

Change History (6)

@ryelle
10 years ago

@ryelle
10 years ago

#1 @ryelle
10 years ago

  • Keywords has-patch added

#2 @iandunn
10 years ago

  • Keywords commit added; has-patch removed
  • Owner set to iandunn
  • Status changed from new to accepted

The patch looks great, thanks Kelly! Adding this to my merge list.

#3 @iandunn
10 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 4345:

WordCamp Coming Soon Page: Enqueue admin bar styles for logged-in users

Fixes #2213
Props ryelle

#4 @iandunn
10 years ago

I removed the bits that registered dashicons and admin-bar, since they were already registered in my tests (dequeue_all_stylesheets() only dequeues them), but let me know if you noticed any conditions where they needed to be registered.

Note: See TracTickets for help on using tickets.