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

add about page #58

Closed
wants to merge 1 commit into from
Closed

add about page #58

wants to merge 1 commit into from

Conversation

jdlrobson
Copy link
Member

added about icon with click event to the welcome page https://bugzilla.wikimedia.org/show_bug.cgi?id=38075
added about page with mock-up content https://bugzilla.wikimedia.org/show_bug.cgi?id=38075
added about page mock-content to the messages-en.properties file

@jdlrobson
Copy link
Member Author

Depends on #59

@jdlrobson
Copy link
Member Author

I have reservations about the localization library changes so it would be good to have this use msg tags

Also we should use something other than ipsum lorem text. The text here - http://www.wikilovesmonuments.org/294/ - seems like a good starting point.

@yuvipanda
Copy link
Contributor

Pinging @dan-nl, because I'm not sure if GH notifications work out there properly

@dan-nl
Copy link
Contributor

dan-nl commented Jul 20, 2012

sorry guys, didn't see this. i'm all for not forking jquery.localize.js within this repo, however, the change does resolve the <option> element issue as well as insuring valid markup, which i believe is important no matter where the app ends up being used, still, i agree, it would be better to fork the original repo and get buy-in there on the changes. anyone know the git url to it?

if necessary, go ahead and use the <msg> elements for now and i'll see if i can get buy-in on the changes to the original repo if it exists.

the link jdlrobson mentions seems like a good place to start with for the actual copy. who usually handles the copy for this app, multichill?

@jdlrobson
Copy link
Member Author

I made some adjustments based on Dan's work. Would be good to get multichill to review but I think it's a good starting point and better than nothing.

@@ -160,6 +160,10 @@ button {
border: 0px;
}

header button {
margin: 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this?

Copy link
Member Author

Choose a reason for hiding this comment

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

overrides browser default margin for buttons

adapted by @jdlrobson based on work by @dan-nl
added about icon with click event to the welcome page <https://bugzilla.wikimedia.org/show_bug.cgi?id=38075>
added about page with suggested content <https://bugzilla.wikimedia.org/show_bug.cgi?id=38075>
added about page suggested content to the messages-en.properties file
required styling fixes
@yuvipanda
Copy link
Contributor

Feel free to merge after adding qqq. Everything else looks ok. The internet gods are not kind to me.

@jdlrobson
Copy link
Member Author

merged

@jdlrobson jdlrobson closed this Jul 26, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants