Added mongo theming #13

Merged
merged 2 commits into from Mar 9, 2013

Projects

None yet

2 participants

@tveatch

Separated the embedded css into an external file for easier modification of
overall theme, as well as to include outside reference to banner image (which
was too large/complex to store as base64 data in php file).

tveatch Added mongoDB-esque theme
Separated the inline css into an external file for easier editing of
overall theme, as well as to include reference to banner image (which
was too large/complex to store as base64 data in php file).
c8dfdc0
@jwage jwage commented on an outdated diff Mar 5, 2013
mongodbadmin.php
@@ -97,16 +98,16 @@ function linkDocumentReferences($mongo, $document)
$refDb = isset($value['$db']) ? $value['$db'] : $_REQUEST['db'];
- $document[$key]['$ref'] = '<a href="'.$_SERVER['PHP_SELF'].'?db='.urlencode($refDb).'&collection='.$value['$ref'].'">'.$value['$ref'].'</a>';
@jwage
jwage Mar 5, 2013

Why are all the urlencode() calls removed?

@tveatch

My apologies, this is my first time with git. I copy/pasted your original code from the browser into my IDE and made the edits. Something must have happened as I didn't remove all of those urlencode statements, it's as if they dropped out on conversion. Do I just re-clone/re-fork your original code then re-commit? Again sorry for the confusion, hoping to be able to help and not be a nuisance.

@jwage
Owner

It looks like you applied your changes to an old version of the code. If you check out the latest version and reapply your changes it should be fine.

tveatch Commit with current version of original authors code
Previous commit used older version. This version includes updated
elements in original authors code + my theming.
99e9c03
@jwage
Owner

👍

@jwage jwage merged commit a31f4ec into jwage:master Mar 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment