-
Notifications
You must be signed in to change notification settings - Fork 68
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
Blank Space Reduced #29
Blank Space Reduced #29
Conversation
Add a screenshot of resume component and pdf generated. |
Best reviewed: with all changes
Optimal code review plan (3 commits squashed)
|
@garimasingh128 Screenshot of pdf generated: |
@garimasingh128 Pls reply if this is fine or you need some more edition? |
you can also expand the purple panel to the whole length. Don't add line breaks. Just make it match the parent screen in terms of height. When we will add some more info in the panel, it will look filled and good. |
@garimasingh128 I have modified the code according the requirements please check now: |
@garimasingh128 Please inform me if the pull request is good for merge or you want some more modifications. |
it looks fine. I haven't reviewed it yet. ill tell once I do so. once u create a pr and get a confirmation, you are free to move to the next issue. |
Description
The problem of blank space at the bottom has been solved by using
tag in DisplayCard.js. Not only it omits the blank space, but now we can see a more clear and organised view in the generated pdf.
Fixes #8
Type of change
Addition of
tag in DisplayCard.js to avoid any occurance of blank space in the bottom.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test has been done by running the project locally and detecting if the spaces and modification done doesn't disrupt the orientation of the pdf generated.
Checklist: