cv icon indicating copy to clipboard operation
cv copied to clipboard

Adding save/export as pdf feature

Open movoid12 opened this issue 2 years ago • 5 comments

Really great resume website! It would be really cool and interesting to export/save the resume as pdf :) I will try to find a way doing that in the end of the resume.

@all: Do you have any other recommended libraries to use instead of react-pdf library or is it good enough?

movoid12 avatar Dec 26 '23 21:12 movoid12

project has special styles for printing so you can just use system print option ie. Cmd +P on Mac and choose save as pdf 🙂

BartoszJarocki avatar Dec 26 '23 22:12 BartoszJarocki

Thanks @BartoszJarocki for your feedback. Yes thats a great option to save the resume as pdf on desktop but in case a user on smartphone or tablet and want to have the resume as pdf it would be useful to have that save button or do you think its not that necessary to implement that feature atm :) ?

movoid12 avatar Dec 27 '23 16:12 movoid12

Thanks @BartoszJarocki for your feedback. Yes thats a great option to save the resume as pdf on desktop but in case a user on smartphone or tablet and want to have the resume as pdf it would be useful to have that save button or do you think its not that necessary to implement that feature atm :) ?

Yes, this seems to be a good idea @movoid12

RaiPeter avatar Dec 28 '23 19:12 RaiPeter

@BartoszJarocki I would like to work on this.

Proposal

  1. Make the command in the footer item note as clickable with a note on that. Some thing like this
<kbd onClick={()=>setOpen(open => !open)} className="inline-flex h-5...">
          <span className="text-xs">⌘</span>J
        </kbd>{" "}
  1. Else, Change the text according to the screensize. using state isMobile

Intrested to get you suggession on this..

SujithThirumalaisamy avatar Dec 29 '23 12:12 SujithThirumalaisamy

I feel like this is covered in #24 and there's an MR adding a button to access the menu on mobile.

I haven't checked it yet, and don't know if saving works on mobile, but the issues seem related.

OmitNomis avatar Dec 30 '23 13:12 OmitNomis