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 <[email protected]>
in order to avoid conflicts in language files

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to be able to redefine it

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Kai Henseler <[email protected]>
Co-authored-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Kai Henseler <[email protected]>
Co-authored-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
extract configuration as human understandable variable

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
it was hidden before

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
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 <[email protected]>
Signed-off-by: Kai Henseler <[email protected]>
Signed-off-by: Kai Henseler <[email protected]>
bromiesTM and others added 27 commits December 16, 2024 14:38
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
it hides the content on scroll

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order for content to stay under the header

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to restyle later

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
according UI

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
wait till help texts are defined

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to be consistent

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
to be visible on white background

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
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