Skip to content
This repository has been archived by the owner on Dec 23, 2017. It is now read-only.

Alpha banner #73

Merged
merged 7 commits into from
Apr 16, 2015
Merged

Alpha banner #73

merged 7 commits into from
Apr 16, 2015

Conversation

noahmanger
Copy link
Contributor

Here's the first implementation of the alpha banner.

  • Adds the alpha message banner
  • Removes the red background on the logo
  • Deletes old logo files
  • New svg logo with outlined letters instead of text
  • Disables autocomplete properties on search forms

Addresses:

Update

  • Just added close state persistence

Noah Manger added 5 commits April 8, 2015 16:41
Now we don’t need the browsers to render the font!
Turning off autocomplete et. al. on the search bars so that mobile
users don’t get annoying suggestions.

Addresses https://github.com/18F/openFEC/issues/500
Adds a nice big, collapsible notice at the top of the page. Still need
to track the state across pages and add the right email address.
@noahmanger noahmanger changed the title [WIP] Alpha banner Alpha banner Apr 14, 2015
This way if the user closes the banner it will stay closed for the
duration of the session. It will reappear the next time they visit.
<?xml version="1.0" encoding="utf-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

We should note somewhere that we should use an svg minifier at some point

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good call. I didn't know there was such a thing. Any favorites? Wanna create an issue?

msecret pushed a commit that referenced this pull request Apr 16, 2015
@msecret msecret merged commit eb949ca into master Apr 16, 2015
@noahmanger noahmanger deleted the alpha-banner branch August 3, 2015 22:47
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.

None yet

2 participants