Aaron Queen

Thanks!

While working on a project, I came across a few more changes that I needed to make, and I believe these would benefit the community. If WP made it easy to submit a pull request, I’d do so.

1) Can you make the return string of aquila_admin_login_logo_title() be a setting?

2) In postToBlog.php, you commented out the submenu global. In addition, can you only update the [0] index of each submenu[‘edit.php’] item IF it already exists. I came across this when I had a role with only access to edit_posts, but in this case, your plugin always adds the “Blog Tags” menu item back in, which puts WP in a weird state and you end up with a PHP NOTICE error. Hard to explain, but if you just wrap each $submenu..[0] override in an isset, example if (isset($submenu[‘edit.php’][16])), I believe this fixed it for me. Also, what is the purpose of the echo ” at the end of aquila_admin_post_to_blog() ?

3) In WP, if you aren’t logged in for a while, you get the interim login overlay that pops up. On this, what I was seeing in the styling was a bit off. I had added the following CSS to login.css to fix that and one other issue.

body.interim-login #login {
	/* timed out login modal */
	top: auto;
	-webkit-transform: none;
	-ms-transform: none;
	transform: none;
}

body.login #login form p {
  color:#000; /* this allows reset password hint to display, currently white on white */
}

4) I wanted the selected admin submenu item to stand out more, so I added some CSS to aquila.css – obviously, this will need to change a bit now that you have color scheme options, but I think the idea is a decent one and really helps it pop without the text looking blurred like it can when bold.

#adminmenu .wp-submenu li.current a, #adminmenu a.wp-has-current-submenu:focus+.wp-submenu li.current a, #adminmenu .wp-has-current-submenu.opensub .wp-submenu li.current a {
	color:#ffee58;
	font-weight:normal;
	background:#888;
}

#adminmenu .wp-submenu li.current a:hover, #adminmenu a.wp-has-current-submenu:focus+.wp-submenu li.current a:hover, #adminmenu .wp-has-current-submenu.opensub .wp-submenu li.current a:hover, #adminmenu .wp-submenu li.current a:focus, #adminmenu a.wp-has-current-submenu:focus+.wp-submenu li.current a:focus, #adminmenu .wp-has-current-submenu.opensub .wp-submenu li.current a:focus {
	color:#ffee58;
	font-weight:normal;
	background:#888;
}

Hope that helps and makes sense. Any questions, just let me know. If there is some way I can just pull request or branch the plugin to provide suggested changed, I’d be open to that.