Skip to content

Slider documentacion#102

Open
JahsoftWr wants to merge 10 commits intoArisGuimera:masterfrom
JahsoftWr:SliderDocumentacion
Open

Slider documentacion#102
JahsoftWr wants to merge 10 commits intoArisGuimera:masterfrom
JahsoftWr:SliderDocumentacion

Conversation

@JahsoftWr
Copy link
Copy Markdown

Creo que ahora esta bien:

El git

/h/AndroidStudioProjects/Kotling/_JahsoftWr/JetpackComposePro (SliderDocumentacion)
$ git add .

/h/AndroidStudioProjects/Kotling/_JahsoftWr/JetpackComposePro (SliderDocumentacion)
$ git status
On branch SliderDocumentacion
Changes to be committed:
(use "git restore --staged ..." to unstage)
renamed: src/content/docs/sliders/slider.md -> src/content/docs/sliders/slider.mdx

/h/AndroidStudioProjects/Kotling/_JahsoftWr/JetpackComposePro (SliderDocumentacion)
$ git commit -m "slider.mdx"
[SliderDocumentacion eef7c60] slider.mdx
1 file changed, 0 insertions(+), 0 deletions(-)
rename src/content/docs/sliders/{slider.md => slider.mdx} (100%)

/h/AndroidStudioProjects/Kotling/_JahsoftWr/JetpackComposePro (SliderDocumentacion)
$ git push origin SliderDocumentacion
Enumerating objects: 11, done.
Counting objects: 100% (11/11), done.
Delta compression using up to 4 threads
Compressing objects: 100% (6/6), done.
Writing objects: 100% (6/6), 561 bytes | 280.00 KiB/s, done.
Total 6 (delta 3), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (3/3), completed with 3 local objects.
To https://github.com/JahsoftWr/JetpackComposePro.git
42e7a65..eef7c60 SliderDocumentacion -> SliderDocumentacion

/h/AndroidStudioProjects/Kotling/_JahsoftWr/JetpackComposePro (SliderDocumentacion)
$

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 4, 2024

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

Name Status Preview Comments Updated (UTC)
open-compose ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2024 2:26pm

```

</TabItem>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Estaba revisando tu codigo porque yo hare el Componente Section Slider y me di cuenta que te faltan cerrar las etiquetas </Tabs> la de la linea 66 y la de la linea 20.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hola gracias por ayudarme, metí el slider.txt que me pasaste, veremos si esta bien.
Lo pase a Astro para ver si me daba error , pero no se si esta mal o no es astro el tipo de código porque viboreo rojo a casi todo.

No se cual es tu experiencia en git, la mía es escasa mas en esto, debería actualizar mi FOLK antes?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No es necesario actualizar tu fork, pero si quieres que tu proyecto este actualizado lo puedes hacer, sobre lo de la implementación con Astro, no funciono el codigo que te envie?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

SI funciono el slider.txt , no dio error , era ese el problema, y code con astro me marca errores de código, pero no me fije si esta bien configurado vs code.

Gracias

@Richi-Mi
Copy link
Copy Markdown
Contributor

Richi-Mi commented Feb 7, 2024

Prueba con este codigo, descargalo, cambiale el nombre a slider.mdx y reemplazalo en el directorio de sliders, a mi me implemento bien tu documentación:

slider.txt

Copy link
Copy Markdown
Owner

@ArisGuimera ArisGuimera left a comment

Choose a reason for hiding this comment

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

Te dejé varios comentarios para solucionar.

Un saludo y muchas gracias!

@@ -0,0 +1,84 @@
---
title: slider (Control deslizante)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Hay que quitar lo que está entre paréntesis


![Imagen del componente slider](../../../assets/sliders/slider-header.webp)

Un Jetpack Compose Slider es un componente que permite al usuario seleccionar un valor continuo entre dos extremos. El valor seleccionado se puede usar para controlar una función en la aplicación.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Hay que quitar "Jetpack Compose" y slider tiene que ir marcado como código para que se vea así Slider


[comment]: <> (La ruta siempre será assets/nombeComponente/componente-header.webp)

![Imagen del componente slider](../../../assets/sliders/slider-header.webp)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Las imágenes tienen que ir en una tabla para que se muestre la imagen de Material y Material 3

Ejemplo: https://www.jetpackcompose.pro/buttons/button/

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

esto es de material porque solo vi material 3

https://m2.material.io/components/sliders


```kotlin frame="terminal"
@Composable
fun SliderComposable() {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Esto no es la definición del componente

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

no entendí, no es la implementación?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Fíjate por ejemplo como es el button https://www.jetpackcompose.pro/buttons/button/

[desde aquí](https://developer.android.com/jetpack/compose/components/slider?hl=en).
:::

### Ejemplos
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Faltan los tabs para Material y Material 3

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

ok

@ArisGuimera
Copy link
Copy Markdown
Owner

Los deploys están fallando, no encuentra la imagen /assets/sliders/slider-header-material.png"

@JahsoftWr
Copy link
Copy Markdown
Author

Los deploys están fallando, no encuentra la imagen /assets/sliders/slider-header-material.png"
Creo que ahí esta bien.

@JahsoftWr
Copy link
Copy Markdown
Author

Ahi lo arregle.

@ArisGuimera
Copy link
Copy Markdown
Owner

Buenas, la tabla de atributos del método está incompleta

@ArisGuimera
Copy link
Copy Markdown
Owner

@JahsoftWr

@JahsoftWr
Copy link
Copy Markdown
Author

@JahsoftWr

Me fijo.

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