Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

Feature/sup 2910 misc skill picker issues #630

Merged
merged 5 commits into from
Dec 30, 2015

Conversation

vikasrohit
Copy link
Contributor

@nlitwin @tladendo @parthshah Please let me know if you guys find anything out of standard.

Fixed mentioned issues with skill-picker. With following exceptions:

  1. Fixed default data science icon for missing data science skill icons. For actual icons, we need to create separate story/ticket.
  2. Text change for mentioning community in instructions, is static i.e. it won't change based on visibility of communities section.

vikasrohit added 4 commits December 28, 2015 17:32
--  Fixed - The iOS community component seems to be different from the tracks components (e.g., Design). The tracks components all have a grey background that turns white when the pill is switched. The iOS community component should behave exactly the same.
-- Fixed - The "Done" button should not activate if only the iOS community is selected. The user must also select a track. (The iOS community is additional to the tracks; it's not a track itself)
-- Fixed - Icons for several Data Science skills appear broken
-- Fixed - Change the wording at the top to say "select the communities and tracks in which..."
@nlitwin
Copy link
Contributor

nlitwin commented Dec 29, 2015

lgtm

-- Updated the instructions text to be generic.
@vikasrohit
Copy link
Contributor Author

Updated the text to be generic as per comment.

@vikasrohit
Copy link
Contributor Author

Thanks @nlitwin for the review.

vikasrohit pushed a commit that referenced this pull request Dec 30, 2015
…-picker-issues

Feature/sup 2910 misc skill picker issues
@vikasrohit vikasrohit merged commit a285984 into dev Dec 30, 2015
@vikasrohit vikasrohit deleted the feature/sup-2910-misc-skill-picker-issues branch January 28, 2016 05:11
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.

2 participants