Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small tweaks to theme UI #8

Merged
merged 6 commits into from May 1, 2012
Merged

Small tweaks to theme UI #8

merged 6 commits into from May 1, 2012

Conversation

aron
Copy link
Contributor

@aron aron commented Apr 30, 2012

As described in #2332.

✓ Fix spacing above page title (i.e. dataset title etc)
✓ Bootstrap 2.0.2 (@zephod was *supposed* to have done this) - should be trivial and gives important bugfixes for data viewer
✓ Fix subnav width on resources pages (does not extend all the way to the right like it should)
✓ Fix spacing of top menu and search box (do not look right)
✓ Group pages:
  ✓ view page: image / logo is really poorly locate and sized, see e.g.  http://datahub.io/group/wikimedia
  ✓ view page: admin heading on RHS sidebar is wrongly indented

This does now have the latest version of Twitter Bootstrap (2.0.3) and should be checked quite carefully for issues.

NOTE: This shouldn't be merged yet, it's just here to serve as documentation for the branch

This was actually an issue with the rest of the page being wider than
the all the others making the navigation look narrower. I've reduced
the width to 940px, this was originally set to 950px by Rufus as a fix
to #1396 but change does not seem to affect the page.
Gave the login text a little more space between it and the line height
and added a bit more padding to the bottom.
@rossjones rossjones merged commit 4bd2e33 into master May 1, 2012
@amercader amercader deleted the feature-2332-theme-ui branch March 4, 2015 13:30
benjaminleighton added a commit to CSIRO-enviro-informatics/ckan that referenced this pull request Nov 29, 2018
…-env to dev-env

* commit '16095d1fd4adbd5207553777d0cc2ebcbf973bc9': (58 commits)
  Fix solr container build after 2.7.3 merge.
  removing redundant config statements
  fixed solr service to build from local dockerfile
  removing debug from damc
  updating compose to fix bug in config using .env
  adding arg to remove orphans
  Removing dev plugin from ckan damc branch
  opening up db ports
  updating licenses with CSIRO licences
  adding a note in README about licenses
  adding the static-content component
  adding support for static content via nginx, namely licenses.json
  update readme
  adding config for ckanext-hierarchy
  updating compose
  Fix migration script for ckan#4042
  Fix coding standards tests
  Update version number for 2.7.3
  Add link to script in changelog
  Add script for fixing issue ckan#4042
  ...
Zharktas pushed a commit that referenced this pull request Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants