Skip to content

feat: lighthouse data-element tag on draftjs#5

Merged
SaraBianchi merged 3 commits intodevelopfrom
lighthouse-draftjs
Nov 30, 2022
Merged

feat: lighthouse data-element tag on draftjs#5
SaraBianchi merged 3 commits intodevelopfrom
lighthouse-draftjs

Conversation

@pnicolli
Copy link
Copy Markdown
Member

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 25, 2022

Bundle size diff

Old size New size Diff
13.1 MB 13.11 MB 13.78 KB (0.10%)

@SaraBianchi
Copy link
Copy Markdown

Aggiustato tooltip per inserimento di link e data element in draftJs

Screenshot 2022-11-29 alle 16 32 13

@Wagner3UB ti aggiungo qui se vuoi fare anche tu test e review.

@Wagner3UB
Copy link
Copy Markdown

@pnicolli @SaraBianchi Per conoscenza:

Ho provato a impostare gli altri stili (bold, italico ecc) un pó a caso e ogni tanto il data-element sparisce. Basta re-impostarlo però che ricompare e si vede perfettamente.

Come è un strumento a uso nostro, interno, basta che Baio e Nicola sappiano di questo piccolissimo bug. Penso anche che non ci saranno mille stile su un link al footer, ma non si sa mai. Non solo, penso che il problema sia impostare gli altri stili dopo impostato data-element

@SaraBianchi SaraBianchi merged commit 3147400 into develop Nov 30, 2022
@pnicolli pnicolli deleted the lighthouse-draftjs branch November 30, 2022 11:44
@SaraBianchi SaraBianchi restored the lighthouse-draftjs branch December 5, 2022 12:04
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