Skip to content

Conversation

@thlehmann-ionos
Copy link

Pragmatic change.

Three problems with the existing spaghetti code:

  1. The Svelte code should not interact with DOM elements across components
  2. The routing is implemented in at least two places
  3. The explicit goto('/') was no-op since the href had already changed the location (thus adding the preventDefault modifier)

== Limitation

Directly navigating to / will still render Chat

For that we probably have to either change every place that explicitly routes to / or, better, change the approach and change / to render our own start page. This would imply bigger changes to the routing in OWU and to our management service too.

Refs: #open-webui#205

thlehmann-ionos and others added 30 commits November 20, 2024 09:38
* Works with podman-compose too.
* Copy .env.example to .env and add proper configs

Based on: https://docs.openwebui.com/getting-started/advanced-topics/development/
Ensure packages are uptodate.

Therefor a wrapper script was added.
They get copied in the Dockerfile during container build, which is a
nuisance during development.
v3.0 prefixed with exos-
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
in order to avoid conflicts in language files

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
in order to be able to redefine it

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Co-authored-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Co-authored-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
extract configuration as human understandable variable

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
it was hidden before

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
exos style v3.0
* cleaned up from other tenants
* removed styling of html tags
* removed variables definition
* prepended css classes with exos- in order no to collide with other styles
sed -E 's/\.(-?[_a-zA-Z]+[_a-zA-Z0-9-]*)/.exos-\1/g' exos.foo.css > exos-prefixed.css
* still uses external fonts!

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
bromiesTM and others added 27 commits December 16, 2024 14:38
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
it hides the content on scroll

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
in order for content to stay under the header

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
in order to restyle later

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
according UI

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
wait till help texts are defined

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
in order to be consistent

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
to be visible on white background

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Pragmatic change.

Three problems with the existing spaghetti code:

1. The Svelte code should not interact with DOM elements across
   components
2. The routing is implemented in at least two places
3. The explicit goto('/') was no-op since the href had already changed
   the location (thus adding the preventDefault modifier)

== Limitation

Directly navigating to / will still render Chat

For that we probably have to either change every place that explicitly
routes to / or, better, change the approach and change / to render our
own start page. This would imply bigger changes to the routing in OWU
and to our management service too.

Refs: #open-webui#205
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.

4 participants