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

fix: refs #7353 horizontal scroll #735

Merged
jorgep merged 7 commits from 7353-hotfix-fixUI into master 2024-09-20 13:49:04 +00:00
Conversation 0 Commits 7 Files Changed 5 +58 -64
jorgep commented 2024-09-20 09:23:29 +00:00
Member
Copy Link
No description provided.
jorgep added 1 commit 2024-09-20 09:23:29 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
ac6d98cd36 fix: refs #7353 horizontal scroll
jorgep added 1 commit 2024-09-20 10:17:15 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e1f3589110 fix: refs #7353 horizontal scroll
jorgep added 1 commit 2024-09-20 11:17:02 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
87efc2f355 fix: refs #7353 responsive filters
jorgep reviewed 2024-09-20 11:17:32 +00:00
src/components/VnTable/VnTable.vue
@ -416,3 +416,3 @@
v-if="$props.rightSearch"
icon="filter_alt"
class="bg-vn-section-color q-ml-md"
class="bg-vn-section-color q-ml-sm"
jorgep commented 2024-09-20 11:17:32 +00:00
Author
Member
Copy Link

Había un espacio demasiado grande

Había un espacio demasiado grande
jorgep reviewed 2024-09-20 11:17:41 +00:00
src/components/VnTable/VnTable.vue
@ -503,3 +503,3 @@
:title="btn.title"
:icon="btn.icon"
class="q-px-sm"
class="q-pa-xs"
jorgep commented 2024-09-20 11:17:41 +00:00
Author
Member
Copy Link

Lo mismo que arriba.

Lo mismo que arriba.
jorgep reviewed 2024-09-20 11:18:14 +00:00
src/components/VnTable/VnTable.vue
@ -751,1 +752,4 @@
}
&__top {
padding: 12px 0px;
jorgep commented 2024-09-20 11:18:14 +00:00
Author
Member
Copy Link

Demasiado espacio.

Demasiado espacio.
jorgep reviewed 2024-09-20 11:18:27 +00:00
src/components/VnTable/VnVisibleColumn.vue
@ -136,3 +136,3 @@
</script>
<template>
<QBtn icon="vn:visible_columns" class="bg-vn-section-color q-mr-md q-px-sm" dense>
<QBtn icon="vn:visible_columns" class="bg-vn-section-color q-mr-sm q-px-sm" dense>
jorgep commented 2024-09-20 11:18:27 +00:00
Author
Member
Copy Link

Demasiado espacio.

Demasiado espacio.
jsegarra commented 2024-09-20 11:43:55 +00:00
Member
Copy Link

y porque no xs? habría menos espacio o demasiado agobiante?

y porque no xs? habría menos espacio o demasiado agobiante?
jorgep reviewed 2024-09-20 11:18:43 +00:00
src/pages/Monitor/MonitorClients.vue Outdated
@ -144,0 +143,4 @@
margin-bottom: 0;
flex-direction: row;
> * {
max-width: 105px;
jorgep commented 2024-09-20 11:18:43 +00:00
Author
Member
Copy Link

Ahora solo se usa el espacio necesario

Ahora solo se usa el espacio necesario
jorgep reviewed 2024-09-20 11:19:27 +00:00
src/pages/Monitor/Ticket/MonitorTickets.vue Outdated
@ -371,3 +365,3 @@
</template>
<template #column-totalProblems="{ row }">
<QTd class="no-padding" style="max-width: 60px">
<span>
jorgep commented 2024-09-20 11:19:27 +00:00
Author
Member
Copy Link

el template ya engloba todo el contenido en un qtd

el template ya engloba todo el contenido en un qtd
jorgep reviewed 2024-09-20 11:19:47 +00:00
src/pages/Monitor/Ticket/MonitorTickets.vue
@ -507,2 +491,4 @@
</VnTable>
</template>
<style lang="scss" scoped>
td .q-icon {
jorgep commented 2024-09-20 11:19:47 +00:00
Author
Member
Copy Link

damos un ligero espacio entre iconos.

damos un ligero espacio entre iconos.
jorgep added 1 commit 2024-09-20 11:20:43 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
3e101242f6 chore: refs #7353 drop useless code
jorgep added 1 commit 2024-09-20 11:22:17 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
7a253d314c chore: refs #7353 drop useless code
jorgep added 1 commit 2024-09-20 11:22:35 +00:00
jorgep added the
CR / Tests passed
 label 2024-09-20 11:41:20 +00:00
jorgep requested review from jsegarra 2024-09-20 11:41:25 +00:00
jsegarra requested changes 2024-09-20 12:11:59 +00:00
Dismissed
jsegarra left a comment
Member
Copy Link

Propongo, dar mas espacio a los VnInputDate en MonitorClients.
Miotivo del cambio: la tabla clientes tien "Sin datos disponibles", mientras que acciones no lo muestra

Propongo, dar mas espacio a los VnInputDate en MonitorClients. Miotivo del cambio: la tabla clientes tien "Sin datos disponibles", mientras que acciones no lo muestra
jorgep added 1 commit 2024-09-20 13:46:59 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
df56eefe2f fix: refs #7353 fine tunning ui
jorgep requested review from jsegarra 2024-09-20 13:47:32 +00:00
jsegarra approved these changes 2024-09-20 13:47:35 +00:00
jorgep merged commit e8beebff32 into master 2024-09-20 13:49:04 +00:00
jorgep deleted branch 7353-hotfix-fixUI 2024-09-20 13:49:04 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
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
2 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#735
No description provided.
Delete Branch "7353-hotfix-fixUI"

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?