Add menu id for admin-bar's dashboard and view-site link

[423300]

Please add menu id for them, So I can use remove_menu to remove them out

[szadok]

My first contribution to WordPress :)
Added "view-site" id and "dashboard" id to the menu.
@nacin, if it is ok - close the bug.

[nacin]

Looks good. Can you check other add_menu calls in the file for missing IDs? I saw at least one.

[szadok]

Replying to nacin:

Looks good. Can you check other add_menu calls in the file for missing IDs? I saw at least one.

Repatched: added also id = "edit-profile" to "Edit My Profile" & id="logout" to "Logout"

[azaozz]

+1. Don't think that can break anything in 3.2.

[ryan]

In [18333]:

Add menu id for admin-bar's dashboard and view-site link. Props szadok. fixes #17873

[JarretC]

Maybe it is just me but the logout and dashboard IDs seem a little bit too basic. The logout ID more than the dashboard one but just thinking of possible CSS styling issues that could happen down the road since IDs are meant to be unique on a page.

Maybe something more along the lines of prefixing the IDs with admin-bar- or similar?

[nacin]

These are just the internal menu IDs. The admin bar API prepends 'wp-admin-bar-' before turning them into HTML IDs.

[JarretC]

Why yes it does...apparently I need to pay attention more!