Core
#71
(10-25-2013, 12:09 AM)xoft Wrote: Show me a comparison, I still believe it would look better detached.
   

(10-25-2013, 12:09 AM)xoft Wrote: Maybe move the MoveUp / MoveDown buttons to the left and make the plugin name column expand, so that all the buttons on the page align on the right side?

I think the length of the column is determinant by the plugin with the longest name, so I don't think I can do much about it.
Reply
Thanks given by:
#72
Add this to the first td or th
Code:
<th width="100%">...</th>
It will stretch the first column to maximum size
Reply
Thanks given by:
#73
Now lose the table around the headers, make them regular <h2> or whatever level is appropriate, and add a sentence to each ("These plugins are currently running", "These plugins encountered an error while starting up", "These plugins are disabled in the configuration")
Reply
Thanks given by:
#74
I don't realy understand everything. Maybe you can edit it yourself and then post the result here so we can compare?
The lines are: 92, 112 and 122(just noticed that there is a tab missing).
Reply
Thanks given by:
#75
Is that the ReWrite branch of the Core project, or some other repo?
Reply
Thanks given by:
#76
The ReWrite branch.
Reply
Thanks given by:
#77
I've tried to make the changes, but I'm not a web designer, so it looks crummy, but I think it captures the idea.
Your version:
   

My version:
   

Also, your rewrite is lacking the console /reload command, and that suxTongue Reloading through the web interface causes crashes for some reason - probably cPluginManager not being thread-safe when reloading plugins.
Reply
Thanks given by:
#78
I'm not sure if that much text is necessaryTongue I mean "Active plugins" > "These plugins are active" really? haha.
But yeah whatever, personally I don't have a preference, I think both versions look good. It would make sense to remove the colored text though, since it's already categorized.
Reply
Thanks given by:
#79
I was hoping someone would come up with something more clever to put there; it makes sense to have the text for Errors and Disabled, and I didn't think it'd look good to have no text for the Active.

I've committed my quick changes as a ReWrite_pluginlist branch, feel free to experiment with it.

I've come up with a nicer text, just for you Smile

   

I question the correctness of the plugin - it displays ErrPlgB in both Active and Errors.
It should be in Errors only, because it's only an empty folder (no file -> no Initialize() function, so it fails to load)
Reply
Thanks given by:
#80
I have to say that it looks much better then I expected Smile I'm now working on the bug where an plugin with an empty folder shows up in both Error and Active.
Reply
Thanks given by:




Users browsing this thread: 2 Guest(s)