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

feat: aggiunto btn per copiare il codice nella clipboard #972

Merged
merged 8 commits into from
Oct 9, 2023

Conversation

Virtute90
Copy link
Contributor

Descrizione

Nuova features per copia il codice nella clipboard con un clip, molto comodo
image

Miglioramenti proposti

Il div contenitore sarebbe da settare in overlay per una migliore visualizzazione.

Checklist

@vercel
Copy link

vercel bot commented Sep 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
bootstrap-italia ✅ Ready (Inspect) Visit Preview Oct 6, 2023 1:40pm

@astagi
Copy link
Member

astagi commented Oct 2, 2023

Grazie @Virtute90 molto utile! secondo te è possibile estenderlo su tutte le parti coding? ad esempio, qua in una c'è nell'altra no..

image

@Virtute90
Copy link
Contributor Author

Grazie @Virtute90 molto utile! secondo te è possibile estenderlo su tutte le parti coding? ad esempio, qua in una c'è nell'altra no..

image

Domani controllo e ti so dire

@Virtute90
Copy link
Contributor Author

Grazie @Virtute90 molto utile! secondo te è possibile estenderlo su tutte le parti coding? ad esempio, qua in una c'è nell'altra no..

image

Il bottone viene aggiunto al codice generato nel template con

{% capture example %}
{% endcapture %}{% include example.html content=example %}

invece non viene considerato quello generato con

> ``` html
> ``` 

Avrei già la modifica da integrare, procedo?

Secondo quello generato con ''' html sono solo esempi e forse non sono da copiare...

@astagi
Copy link
Member

astagi commented Oct 4, 2023

Grazie @Virtute90 molto utile! secondo te è possibile estenderlo su tutte le parti coding? ad esempio, qua in una c'è nell'altra no..
image

Il bottone viene aggiunto al codice generato nel template con

{% capture example %}
{% endcapture %}{% include example.html content=example %}

invece non viene considerato quello generato con

> ``` html
> ``` 

Avrei già la modifica da integrare, procedo?

Secondo quello generato con ''' html sono solo esempi e forse non sono da copiare...

Grazie @Virtute90 sisi procedi pure a questo punto includiamo anche quegli snippet, male non fa :)

@Virtute90
Copy link
Contributor Author

Grazie @Virtute90 molto utile! secondo te è possibile estenderlo su tutte le parti coding? ad esempio, qua in una c'è nell'altra no..
image

Il bottone viene aggiunto al codice generato nel template con

{% capture example %}
{% endcapture %}{% include example.html content=example %}

invece non viene considerato quello generato con

> ``` html
> ``` 

Avrei già la modifica da integrare, procedo?
Secondo quello generato con ''' html sono solo esempi e forse non sono da copiare...

Grazie @Virtute90 sisi procedi pure a questo punto includiamo anche quegli snippet, male non fa :)

Aggiunto commit

@astagi
Copy link
Member

astagi commented Oct 5, 2023

Grazie mille @Virtute90 ! Riguardo entro stasera, prima di fare la prossima release

Copy link
Member

@astagi astagi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Virtute90 due cose di review:

  • Farei un'unica funzione di copia per non avere il codice duplicato
  • Invece di Copy code to clipboard mettere in italiano Copia il codice negli appunti

Grazie mille!

@Virtute90
Copy link
Contributor Author

Virtute90 commented Oct 6, 2023

@Virtute90 due cose di review:

  • Farei un'unica funzione di copia per non avere il codice duplicato
  • Invece di Copy code to clipboard mettere in italiano Copia il codice negli appunti

Grazie mille!

Fatto!

ps: come posso contattarti? dovrei chiederti una cosa 😅

@astagi
Copy link
Member

astagi commented Oct 6, 2023

@Virtute90 due cose di review:

  • Farei un'unica funzione di copia per non avere il codice duplicato
  • Invece di Copy code to clipboard mettere in italiano Copia il codice negli appunti

Grazie mille!

Fatto!

ps: come posso contattarti? dovrei chiederti una cosa 😅

Grazie mille @Virtute90 ! :) Certamente, mi trovi come Andrea Stagi sullo Slack pubblico di Developers Italia 👍

@astagi
Copy link
Member

astagi commented Oct 6, 2023

@Virtute90 ho fatto un piccolo fix per ripristinare il bottone dopo 5 secondi e spostato in alto a destra, che ne dici?

@Virtute90
Copy link
Contributor Author

@Virtute90 ho fatto un piccolo fix per ripristinare il bottone dopo 5 secondi e spostato in alto a destra, che ne dici?

Ho visto, mi piace il ripristino del bottone ed è più allineato al box!

@astagi astagi merged commit a0c505b into italia:main Oct 9, 2023
@Virtute90 Virtute90 deleted the feat/add-code-clipboard branch October 9, 2023 09:17
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.

2 participants