Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Interview page #131

Merged
merged 29 commits into from
Feb 21, 2015
Merged

New Interview page #131

merged 29 commits into from
Feb 21, 2015

Conversation

FadyKalo-zz
Copy link
Collaborator

  • check no-interview.( HIGH)
  • animation in the hover, staging has a better effect.
  • Report score ( HIGH)
  • uniform the action icon
  • change label to mylibrary change library to my library #137
  • correct reminder in settings
  • add empty billing settings page.
  • uniform colors
  • center and reduce the size of the list view in the report page ( HIGH)
  • remove background color from time box list view ( HIGH)
  • make sure open interview, today interviews and interview fit correctly in the page. ( add screen shot)( HIGH)
  • fix missing icons in local environment
  • fix list layout
  • text truncation for long names and jobs

@FadyKalo-zz FadyKalo-zz self-assigned this Feb 14, 2015
@giuse88
Copy link
Owner

giuse88 commented Feb 15, 2015

The list view doesn't work. Did you forgot to do commit some changes?

Staging over animation :

 .interview-item .header {
      -webkit-transition: all 0.3s;
      -moz-transition: all 0.3s;
      -ms-transition: all 0.3s;
      -o-transition: all 0.3s;
      transition: all 0.3s;
  }

@fadykalo

@FadyKalo-zz
Copy link
Collaborator Author

Thanks, for the .hover style, added to now.

@giuse88
Copy link
Owner

giuse88 commented Feb 16, 2015

The list view has a problem when you show the name of the candidate.

screen shot 2015-02-16 at 08 45 48

FadyKalo-zz pushed a commit that referenced this pull request Feb 21, 2015
@FadyKalo-zz FadyKalo-zz merged commit 46b3e0d into master Feb 21, 2015
@FadyKalo-zz FadyKalo-zz deleted the new-page branch February 21, 2015 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants