Skip to content
This repository has been archived by the owner on Oct 5, 2020. It is now read-only.

small ui fixes for 3 column theme #343

Merged
merged 5 commits into from
May 25, 2016

Conversation

janmichaelyu
Copy link
Contributor

@janmichaelyu janmichaelyu commented May 25, 2016

search results right margin
details for json bottom margin
details for binaries bottom margin

janmichaelyu and others added 4 commits December 11, 2015 15:57
search results need right margin
details for json need bottom margin
details for binaries need bottom margin
@janmichaelyu
Copy link
Contributor Author

@grtjn @jenbreese Please review.

@@ -8,13 +8,14 @@
<pre ng-if="ctrl.type!=='json'" class="pre-xml">{{ctrl.xml}}</pre>

<a ng-if="ctrl.type==='binary'" target="_blank" ng-href="/v1/documents?uri={{ctrl.uri}}&transform=download" class="btn btn-default">Download</a>

<p>&nbsp;</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we fix this with CSS instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed to CSS implementation

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will look at this shortly

@grtjn
Copy link
Contributor

grtjn commented May 25, 2016

@jenbreese Can you review the CSS?

@jenbreese
Copy link
Contributor

@grtjn and @janmichaelyu : it looks good to me. I tested it.

@grtjn
Copy link
Contributor

grtjn commented May 25, 2016

Taking @jenbreese's word for it.. ;)

@grtjn grtjn merged commit 7c6fdf1 into marklogic-community:ui-themes May 25, 2016
@janmichaelyu janmichaelyu deleted the 3-column-ui-theme branch May 26, 2016 02:52
@grtjn grtjn modified the milestone: 1.1.0 May 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants