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

Cv form UI improvement #258

Closed
4 tasks done
TiSiE opened this issue Jul 20, 2016 · 1 comment
Closed
4 tasks done

Cv form UI improvement #258

TiSiE opened this issue Jul 20, 2016 · 1 comment
Assignees
Milestone

Comments

@TiSiE
Copy link
Member

TiSiE commented Jul 20, 2016

The form for creating and editing a resume needs to be visually improved.

  • Move the remove button next to the edit button
    move-remove-button
    That means removing an item is only possible in summary view, but this behaviour is INTENDED.
  • Make "Desired Employment" and "Native Language" a SummaryForm.
  • Improve Employment History:
    Form:
    employment-history-form
    Summary:
    employment-history-summary
  • Education History: Same as above, hide end date if "ongoing" is checked and prettify date range in summary view
@TiSiE TiSiE added this to the v0.26 milestone Jul 20, 2016
fedys added a commit to fedys/YAWIK that referenced this issue Jul 25, 2016
* fixed form collection removal right after an insertion (without
refreshing a browser) when summaryForm is used
* form collection removal buttons moved into summary forms
* created \Core\Form\EmptySummaryAwareTrait
* improved summary form rendering of select with multiple options
without an <optgroup>
refs cross-solutiongh-258
fedys added a commit to fedys/YAWIK that referenced this issue Jul 25, 2016
* fixed form collection removal right after an insertion (without
refreshing a browser) when summaryForm is used
* form collection removal buttons moved into summary forms
* created \Core\Form\EmptySummaryAwareTrait
* improved summary form rendering of select with multiple options
without an <optgroup>
refs cross-solutiongh-258
fedys added a commit to fedys/YAWIK that referenced this issue Jul 26, 2016
@TiSiE
Copy link
Member Author

TiSiE commented Jul 26, 2016

Implemented with #261

@TiSiE TiSiE closed this as completed Jul 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants