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

Redesign & modernise CSS, add Google Analytics option, other miscellaneous improvements. #1

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

capncodewash
Copy link

This fork of the Kuali version of SchemaSpy adds the following features and enhancements:

  • A modern stylesheet for screen and print based on Foundation (no more lime green - yay!)
  • Responsive design, including the removal of many layout tables.
  • The '-showanomalies' flag, which makes the 'anomalies' tab visible. The default is now false.
  • Make comments visible by default
  • The '-ga' flag, which lets you specify a Google Analytics tracking code to include in all output
  • The '-showdbname' flag, which shows the database schema name in table descriptions. The default is now false.
  • The donation links have been removed. There is still a link to the SchemaSpy website.

A screenshot is visible in the README file of my fork.

Thanks,

Graeme West

… in table pages. Other form checkboxes TODO.
…tion tab bar. Removed unnecessary styles and cleaned up structure. Removed 'SchemaSpy' from the description of each element, and replaced it with a single footer element linking to the SchemaSpy website.
…ML syntax. Removing some of the structural tables.
… the contents fill the window. Hide the HREF attribute in the print stylesheet. Fix more form element labels.
…king ID to be placed onto each page in the HTML output.
@leoherbie
Copy link
Contributor

Thanks for your PR. We are taking a look at it. Hopefully we will have some feedback soon.

@capncodewash
Copy link
Author

Hi Travis, do you have any comments on my changes?

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.

2 participants