verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 91 Releases Wiki Activity

#7404 - fixSomeStyleIssues #783

Closed
jsegarra wants to merge 20 commits from 7404-fixSomeStyleIssues into dev
pull from: 7404-fixSomeStyleIssues
merge into: verdnatura:dev
Conversation 4 Commits 20 Files Changed 7 +172 -130
jsegarra commented 2024-10-01 12:10:48 +00:00
Member
Copy Link
No description provided.
jsegarra added 5 commits 2024-10-01 12:10:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
86ab73eeb2 fix: refs #7404 ItemFixedPrice
jsegarra changed title from 7404-fixSomeStyleIssues to WIP: 7404-fixSomeStyleIssues 2024-10-01 12:18:26 +00:00
jsegarra requested review from pablone 2024-10-01 12:18:33 +00:00
jsegarra commented 2024-10-01 12:19:23 +00:00
Author
Member
Copy Link

He puesto a VnSelect la opción del hover, sin embargo en routeExtendedList, creo que por el tamaño de la columna, hace un salto de tamaño

He puesto a VnSelect la opción del hover, sin embargo en routeExtendedList, creo que por el tamaño de la columna, hace un salto de tamaño
jsegarra added 2 commits 2024-10-01 19:49:33 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
64bb27ca7f style: refs #7404 fix RouteExtendedList
jsegarra added 1 commit 2024-10-02 12:56:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6c3c7a3913 style: refs #7404 #7404 fix ItemListFilter
jsegarra added 1 commit 2024-10-02 22:08:53 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
22ec143ee0 Merge branch 'dev' into 7404-fixSomeStyleIssues
jsegarra added 1 commit 2024-10-02 22:13:22 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6e16febe8b style: refs #7404 #7404 remove comments
jsegarra requested review from alexm 2024-10-02 22:13:51 +00:00
jsegarra changed title from WIP: 7404-fixSomeStyleIssues to 7404-fixSomeStyleIssues 2024-10-02 22:13:54 +00:00
jsegarra changed title from 7404-fixSomeStyleIssues to #7404 - fixSomeStyleIssues 2024-10-02 22:14:03 +00:00
pablone requested changes 2024-10-04 09:26:22 +00:00
pablone left a comment
Member
Copy Link

Revisar los iconos de la searchBar y de los filtros

Revisar los iconos de la searchBar y de los filtros
jsegarra added 4 commits 2024-10-04 22:27:21 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
47ea8d1e5b perf: refs #7404 #7404 clean style rules
jsegarra commented 2024-10-04 22:29:22 +00:00
Author
Member
Copy Link

Revisar los iconos de la searchBar y de los filtros

Dejo adjuntadas unas capturas con el resultado
image
image
image

Cualquier cosa, comentadme

> Revisar los iconos de la searchBar y de los filtros Dejo adjuntadas unas capturas con el resultado ![image](/attachments/7ce9fd84-00a6-42c1-bbf9-9771a9402f7b) ![image](/attachments/7ab89e50-8cab-4efa-b224-479b7921a643) ![image](/attachments/267dbb0c-db5f-439d-b027-505d36f79bde) Cualquier cosa, comentadme
image.png
60 KiB
image.png
68 KiB
image.png
52 KiB
jsegarra requested review from pablone 2024-10-04 22:29:31 +00:00
alexm requested changes 2024-10-07 11:09:00 +00:00
Dismissed
src/components/common/VnSelect.vue Outdated
@ -38,10 +37,6 @@ const $props = defineProps({
type: [Array],
default: () => [],
},
exprBuilder: {
alexm commented 2024-10-07 11:08:51 +00:00
Member
Copy Link

Ya no se quiere?

Ya no se quiere?
jsegarra commented 2024-10-07 11:20:59 +00:00
Author
Member
Copy Link

Y las valdiaciones tampoco? WTF?
Reviso.
Gracias

Y las valdiaciones tampoco? WTF? Reviso. Gracias
src/pages/Route/RouteExtendedList.vue Outdated
@ -87,6 +87,7 @@ const columns = computed(() => [
},
},
columnClass: 'expand',
style: 'min-width: 210px;',
alexm commented 2024-10-07 11:08:35 +00:00
Member
Copy Link

Antes que ir poniendo con pixeles el tamaño nimo o maximo miraria de hacer algo standard

Antes que ir poniendo con pixeles el tamaño nimo o maximo miraria de hacer algo standard
jsegarra commented 2024-10-07 12:17:20 +00:00
Author
Member
Copy Link

La columna ya tiene la clase expand.
Lo que ocurre es que como el select muestra o no el icono de la x, le falta mas espacio y por tanto cada vez que haces hover sobre el input te genera un cambio visual molesto. Es por eso que le he puesto mas tamaño.

Podemos darle una vuelta o añadir esta propiedad CSS a la clase expand

La columna ya tiene la clase expand. Lo que ocurre es que como el select muestra o no el icono de la x, le falta mas espacio y por tanto cada vez que haces hover sobre el input te genera un cambio visual molesto. Es por eso que le he puesto mas tamaño. Podemos darle una vuelta o añadir esta propiedad CSS a la clase expand
jsegarra added 2 commits 2024-10-07 11:24:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
36ce685d6e fix: refs #7404 #7404 VnSelect resolve conflcits
jsegarra reviewed 2024-10-07 11:24:36 +00:00
src/components/common/VnSelect.vue Outdated
@ -226,4 +227,2 @@
return value ?? true;
}
const getVal = (val) => ($props.useLike ? { like: `%${val}%` } : val);
jsegarra commented 2024-10-07 11:24:36 +00:00
Author
Member
Copy Link

Se ha movido a la linea 107

Se ha movido a la linea 107
jsegarra added 3 commits 2024-10-07 11:59:38 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
68f8037db4 fix: refs #7404 #7404 VnSelect resolve conflicts
jsegarra requested review from alexm 2024-10-07 11:59:56 +00:00
alexm requested changes 2024-10-07 12:05:51 +00:00
alexm left a comment
Member
Copy Link

Antes que ir poniendo con pixeles el tamaño minimo o maximo para cada columna miraría de hacer algo standard, como lo de expand o shrink

Antes que ir poniendo con pixeles el tamaño minimo o maximo para cada columna miraría de hacer algo standard, como lo de expand o shrink
jsegarra added 1 commit 2024-10-07 18:05:10 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5e75bb29fb revert: refs #7404 #7404 VnSelect hover
jsegarra commented 2024-10-07 18:06:26 +00:00
Author
Member
Copy Link

Hacer el evento hover del VnSelect, no está dentro de eta tarea, asi hago un revert de los cambios para que la PR sea lo mas fiel al nombre de la misma

Hacer el evento hover del VnSelect, no está dentro de eta tarea, asi hago un revert de los cambios para que la PR sea lo mas fiel al nombre de la misma
jsegarra requested review from alexm 2024-10-07 18:06:30 +00:00
pablone refused to review 2024-10-08 05:54:00 +00:00
pablone commented 2024-10-10 12:40:03 +00:00
Member
Copy Link

no entiendo que tiene que ver la tarea 7404 con esta pr

no entiendo que tiene que ver la tarea 7404 con esta pr
jsegarra closed this pull request 2024-10-10 12:41:43 +00:00
All checks were successful
gitea/salix-front/pipeline/pr-dev This commit looks good
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
pablone
alexm
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#783
No description provided.
Delete Branch "7404-fixSomeStyleIssues"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?