491093 - Add careers buttons page to mozilla.com

Status: RESOLVED WONTFIX

(Websites :: careers.mozilla.org, defect)

Description

[Michael Morgan [:morgamic]]

Dan wanted a buttons page that people could view to copy/paste buttons onto external websites and get more traffic to our careers/recruiting page.

Comment 1

[Michael Morgan [:morgamic]]

Attachment: v1, button page

v1, let me know what you guys think

Comment 2

[Reed Loden [:reed]]

Can we put the images under img/ or en-US/img/?

Comment 3

[Wil Clouser [:clouserw]]

Comment on attachment 375430 [details] [diff] [review]
v1, button page

If you can check in the images it would help.  r- though because:

1)  height is misspelled in the for loop and you're missing a quote

2) all the stuff in <textarea> should be htmlspecialchar()'d

3) For some reason the 3rd image down won't show me the alt-text but I have no idea why.  Not blocking on that.

Comment 4

[Michael Morgan [:morgamic]]

Attachment: v2, fixed typo, htmlspecialchars and typo

Will add some images in a second.

Comment 5

[Michael Morgan [:morgamic]]

Attachment: images for buttons

Comment 6

[Wil Clouser [:clouserw]]

Comment on attachment 376260 [details] [diff] [review]
v2, fixed typo, htmlspecialchars and typo

The code looks fine, we just need the images under /en-US/img/careers/

Comment 7

[Steven Garrity [:sgarrity]]

morgamic, is this still relevant?

Comment 8

[Ben (:bensternthal)]

I do not think this is valid anymore. If it is we can re-open but for now I will close as wontfix.

I'll cc Mat on here so at least he is aware.