Lighthouse draftjs #613
Lighthouse draftjs #613
Conversation
Bundle size diff
|
|
|
||
| import DraftJsDropdownButton from './DraftJsDropdownButton'; | ||
|
|
||
| const AlignButtonComponent = (props) => { |
There was a problem hiding this comment.
C'è un refuso, dai un nome specifico a questo componente
| export const AlignButton = injectLazyLibs(['draftJsCreateBlockStyleButton'])( | ||
| AlignButtonComponent, | ||
| ); | ||
|
|
||
| export default React.memo(AlignButton); |
There was a problem hiding this comment.
Idem qua come sopra, occhio quando copi e incolli!
| import React from 'react'; | ||
| import { injectLazyLibs } from '@plone/volto/helpers/Loadable/Loadable'; | ||
| import Icon from '@plone/volto/components/theme/Icon/Icon'; | ||
| import showSVG from '@plone/volto/icons/show.svg'; |
There was a problem hiding this comment.
Questa icona dell'occhio non mi convince perchè viene utilizzata in generale per mostrare e nascondere un contenuto, non vorrei che venisse fraintesa, suggerisco di utilizzare una di queste due icone
- un target, più generico
- l'icona delle news che potrebbe vagamente assomigliare al faro di lighthouse però ditemi voi
src/config/RichTextEditor/config.js
Outdated
| const renderDataElement = (element) => { | ||
| if (typeof document !== 'undefined') { | ||
| document.querySelector(`.lighthouse_${element} a`) && | ||
| document | ||
| .querySelector(`.lighthouse_${element} a`) | ||
| .setAttribute('data-element', element); | ||
| } | ||
| }; |
There was a problem hiding this comment.
Questo seleziona un elemento con una classe specifica ma se creo due link con lo stesso data element il secondo è vuoto, qui ho inserito nel secondo e nel terzo link lo stesso data element, il secondo non popola neanche la classe
Chiedo, non è possibile invece di utilizzare un selettore, applicare direttamente il data element ad un tag <a> invece che a un <p>?
US33941
Added a button to draftjs toolbar to select a data-element inside a link. This option is showed only when both option, "link" and "lighthouse", are selected
Select options:
"Prenotazione appuntamento" -> produce sul link data-element="appointment-booking"
"Domande frequenti" -> produce sul link data-element="faq"
"Segnalazione disservizio" -> produce sul link data-element="report-inefficency"
"Dichiarazione di accessibilità" -> produce sul link data-element="accessibility-link"
"Informativa privacy" -> produce sul link data-element="privacy-policy-link"
To add the data-element to the link it was necessary add a function to push it inside the tag because the constructor of this element wasn't accessible.