Skip to content

add names to indices list and improve layout for better display #17

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

javierariashe
Copy link

Se agregó un input para mostrar y editar el nombre de cada índice en la lista de índices de cada tabla.

Se reutilizó el mismo input que se muestra en las opciones de cada índice. Además, se aplicó el mismo estilo utilizado en la visualización de los campos de las tablas, usando los componentes Row y Col de la biblioteca semi-ui.

Cada elemento se agrupó en un componente Col y todos están dentro de un contenedor Row.

Imagen de antes del cambio:

image

Imagen después del cambio:

image

Copy link

@pabloem pabloem left a comment

Choose a reason for hiding this comment

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

LGTM.
Recomiendo una revision de otro miembro del equipo.

@javierariashe
Copy link
Author

Se modificó el layout de los indices. Ahora el nombre se encuentra arriba seguido de la opción para indicar si es unique. Abajo se encuentran la selección de los campos para el índice y el botón de eliminar. Se eliminó el botón de 'más opciones'.

image

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