Skip to content

Commit

Permalink
fix scroll for the chattyUI, navigation still does not take full heig…
Browse files Browse the repository at this point in the history
…ht if no session is selected

Signed-off-by: Julien Veyssier <[email protected]>
  • Loading branch information
julien-nc committed May 28, 2024
1 parent 7f6742b commit 7c9fe7d
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 16 deletions.
7 changes: 4 additions & 3 deletions src/components/AssistantFormInputs.vue
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,7 @@
@update:value="onUpdateCopywriter" />
</div>
</div>
<div v-else-if="selectedTaskTypeId === 'chatty-llm'" class="assistant-inputs" style="margin-bottom: 0 !important;">
<ChattyLLMInputForm />
</div>
<ChattyLLMInputForm v-else-if="selectedTaskTypeId === 'chatty-llm'" class="chatty-inputs" />
<div v-else class="assistant-inputs">
<div v-if="selectedTaskTypeId === 'OCA\\ContextChat\\TextProcessing\\ContextChatTaskType'" class="assistant-inputs">
<NcCheckboxRadioSwitch :checked.sync="sccEnabled" @update:checked="onUpdateContextChat">
Expand Down Expand Up @@ -318,6 +316,9 @@ export default {
</script>
<style lang="scss" scoped>
.chatty-inputs {
margin-top: 8px;
}
.assistant-inputs {
margin-bottom: 1rem;
Expand Down
5 changes: 4 additions & 1 deletion src/components/AssistantTextProcessingForm.vue
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,7 @@ export default {
<style lang="scss" scoped>
.assistant-form {
width: 100%;
//width: 100%;
display: flex;
flex-direction: column;
align-items: center;
Expand All @@ -358,7 +358,10 @@ export default {
overflow-x: hidden;
.task-input-output-form {
display: flex;
flex-direction: column;
width: 100%;
overflow: auto;
}
.output {
Expand Down
16 changes: 7 additions & 9 deletions src/components/AssistantTextProcessingModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -158,13 +158,9 @@ export default {
</script>
<style lang="scss">
// this is to avoid scroll on the container and leave it to the result block
.assistant-modal .modal-container {
display: flex !important;
//&__content {
//padding: 16px;
//}
// TODO fix this in nc/vue
.modal-container__content .assistant-modal--wrapper {
height: 100%;
}
</style>
Expand All @@ -178,13 +174,14 @@ export default {
}
.assistant-modal--wrapper {
//width: 100%;
padding: 16px;
width: 100%;
display: flex;
overflow-y: auto;
}
.assistant-modal--content {
width: 100%;
padding: 16px;
display: flex;
flex-direction: column;
align-items: center;
Expand All @@ -201,6 +198,7 @@ export default {
.form {
width: 100%;
height: 100%;
}
}
</style>
4 changes: 2 additions & 2 deletions src/components/ChattyLLM/ChattyLLMInputForm.vue
Original file line number Diff line number Diff line change
Expand Up @@ -517,9 +517,9 @@ export default {
<style lang="scss" scoped>
.container {
overflow: auto;
display: flex;
height: 44em;
margin-top: 8px;
height: 100%;
:deep .app-navigation-new {
padding: 0.5em 0 !important;
Expand Down
2 changes: 1 addition & 1 deletion src/components/ChattyLLM/NoSession.vue
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ export default {
<style lang="scss" scoped>
.empty-view {
padding: calc(var(--default-grid-baseline) * 4);
height: 30em;
height: 200px;
}
.empty-content {
Expand Down

0 comments on commit 7c9fe7d

Please sign in to comment.