924548 - Position being viewed should not show up in the current open positions list

Status: VERIFIED FIXED

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

Description

[raymond [:retornam] (inactive)]

STR:
1) Visit http://careers-dev.allizom.org/en-US/position/oJuUXfwO
2) Notice that the position being view is showed under the open positions section for the uroup the position is under
3) I think it will be cleaner to remove an open position from the list when we are viewing that particular position.

Comment 1

[Osmose [:osmose, :mkelly]]

My original instinct was to suggest we mark it instead of removing it, but the location of the current position in the list is pretty unimportant.

Ben: Thoughts? I should be able to handle either option depending on the styling, but I'd favor removal or no change.

Comment 2

[Ben (:bensternthal)]

Attachment: MOZI127_09-Careers_Listings2.jpg

Looks like this was a design oversite. 

The live site has the selected job unlinked and bold. In the new design.. this really wont work. See attached screenshot of what HDSF provided.

https://careers.mozilla.org/en-US/position/oEYTXfwc

We can't just bold the job as the column header is distinguished via bolding. We can't use unlinking as a visual clue - links are only underlined on hover.

Given this I would say we do the removal.

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/lumbergh

https://github.com/mozilla/lumbergh/commit/1ee0d7c82be7bdf02c0418fa2f987e79f76efd0f
Fix bug 924548: Remove current position list in details page.

Comment 4

[Ben (:bensternthal)]

This looks to have been reverted somewhere.

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/lumbergh

https://github.com/mozilla/lumbergh/commit/1617583d1a8ae2bfb617dea52aacab351a5235a3
Re-Fix Bug 924548: Remove current position list in details page.

https://github.com/mozilla/lumbergh/commit/74b715978742a281641116bd83c9406cc22f36ef
Merge pull request #75 from bensternthal/listings-fix

Re-Fix Bug 924548: Remove current position list in details page.

Comment 6

[Ben (:bensternthal)]

checked on dev.. it be there.