Makefile: add dsc target to build debian source packages

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

macros: update dummy pwt implementations so that the doc tests succeed

adds the updated trait definitions and adds a dummy `oncklick`
implementation so that the doc tests succeed again

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

macros: update wrong_comp test to adjust to new additional traits

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

macros: update no_clone test to adjust to new additional traits

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

macros: update compiler version in invalid_type test

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

pwt-macros: fix typo in changelog

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

macros: fix build for tests

it seems adapting the test case in pwt-macros was forgotten at some
point. this makes the test cases build-able again. please note that
this does not make all test cases here work, as some of them seem to
expect the output of specific rustc versions. this should probably be
addressed in a separate commit.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

tree-wide: use `?` instead of match statement where easily possible

this makes the code more concise and fixes the clippy lint
`question_mark` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#question_mark

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

bump version to 0.4.7

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

widget: confirm_dialog: refactor to use message box

this allows using an icon next to the message and generally simplifies
the code.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

input_panel: cleanup: avoid generic parameter to reduce code size

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

menu: remove MenuBar struct

this only had one use, create a Menu where `menubar` was already set to
`true`. use a new constructor instead called `new_menubar()` that does
the same without creating a superfluous type. it's also more idiomatic
to it this way. this fixes the clippy lint `new_ret_no_self` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#new_ret_no_self

pwt-macros: indent list items in doc string properly

avoid the clippy lint `doc_lazy_continuation` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation

dom: use `into()` when converting JSValue to HtmlElement

this cannot fail, so just use `into()` and then wrap with `Some()` to
conform to function signature. this removes the clippy lint
`unnecessary_fallible_conversions` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_fallible_conversions

selection/store/tree_store: use `where` for generics in hooks

instead of in-lining them in the function definition. the `hook!` macro
will add a `'hook` liftime via a `where` clause to them when expanded,
which triggers the clippy lint `multiple_bound_locations` [1] as the
generics would then be bound in the function signature and the `where`
clause. so just define them in the `where` clause right away to avoid
that.

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#multiple_bound_locations

theme: use `is_err` instead of `if let Err(_)`

this is more idiomatic and removes the clippy lint
`redundant_pattern_matching` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching

snack_bar_manager: use a box for `Show` message

the `Show` message takes a whole `SnackBar` struct that is
significantlly larger than the other enum variants here. use a `Box` to
improve performance and remove the clippy lint `large_enum_variant` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#large_enum_variant

page_stack/page_view: use `Ord` trait to get rid of comparison chains

this makes the code more readable and idiomatic, fixes the clippy lint
`comparison_chain` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#comparison_chain

form context: allow register_field with too many arguments

this is a publically facing api so allow the clippy lint
`too_many_arguments` [1] here for now and add a to-do to possible
refactor this to use it's own parameter types.

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments

form module: allow module inception as we only public re-expose `Form`

allowing the clippy lint `module_inception` [1] here makes some sense as
no one should every really need to import anything but `pwt::form::Form`
or `pwt::form::PwtForm`.

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#module_inception

navigation_drawer: use if-let instead of map to avoid unit-closure map

an if-let statement is more clear and simplifies the code here quite a
bit. also fixes the clippy lint `option_map_unit_fn` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#option_map_unit_fn

split_pane: take ownership of `self` in `PaneSize` `to_*` functions

this breaks the self convention for types that are `Copy` such as
`PaneSize`. this should improve readability and fixes the
`wrong_self_convention` clippy lint [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention

loader/data_table: refactor match statements that return booleans

this can usually be more succinctly expressed with the `matches!` macro
or other methods to deal with `Result`s or `Option`s. this also fixed
the clippy lint `match_like_matches_macro` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro

data_table/menu: re-write loops as while-let loops

this is more idiomatic and fixes the clippy lint `while_let_loop` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#while_let_loop

tree wide: allow type complexity lints where necessary

allow type complexity where neccessary. this avoid triggering the clippy
lint `type_complexity` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity

theme_mode_selector: `_`-prefix ctx as it's only used in recursion

since we olny pass through the `ctx` parameter and don't use it here
ourselves mark it with a `-`-prefix. this fixes the clippy lint
`only_used_in_recursion` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#only_used_in_recursion

tree wide: remove empty lines between doc comments and code

to make clear that the doc comments actually do document the following
lines of code. this fices the clippy lint
`empty_line_after_doc_comments` [1].

[1]:
https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments

tree wide: apply auto-appliable clippy fix-ups

this mostly includes:

- switching from `Into` to `From` implementations as that also provides
  an `Into` implementation for "free"
- adding `Default` implementations where possible (e.g. if a `new`
  function without parameters exists)
- removing unnecessary reference and dereference operations (e.g. if the
  compiler would do that automatically anyway)
- using more idiomatic syntax (e.g. using `contains` instead of `<` and
  `>=` operators)
- removing closures where they aren't necessary
- removing `format!` macros where `to_string()` would do to improve
  performance
- remove unnecessary convertions and into calls

these should not change the semantics of the exists code itself, so
there are no breaking changes. however, adding `Default` implementations
and such, does add new public APIs for some types.

gettext_wrapper: run rustfmt

widget: dropdown: update picker placement after showing

It seems that under certain circumstances (e.g. a dropdown in a
dialog) updating the position of the picker before it's opened (it's a
popover) does not work properly in safari/webkit/epiphany.

When trying to open such a dropdown, it (or some parts of it) would be
rendered with a height of 0px, even though the elements have a height
set explicitly.

To fix this, update the picker placement after opening it. What
exactly is the cause (e.g. if it's a browser bug or our css/aligning)
is yet to be determined.

This fixes an issue on webkit (tested with epiphany) that opening
dropdowns in a dialog did not show the dropdown properly.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Tested-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
 [ TL: extend comment to mention webkit explicitly ]
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 0.4.6

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

whitepaper: fix some typos

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

widget: input panel: use correct column for custom childs

we added a 'padding column' recently, and we forgot to adapt
the custom childs for that.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

whitepaper: re-flow text to 80 cc width

Wrap at a consistent text-width of 80 character columns, that is
a relatively standard width for prose and a good balance for having
more context visible at the same time while not requiring to much neck
moving.

Also fix a unclosed code-block by adding the third backtick

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

readme: re-wrap to 80cc and small fixes

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 0.4.5

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

widget: display: don't set inital value

when doing that, a new value that is set via `new` won't get used as a
value anymore, which is undesired. only use the value for the default,
which was the original intenion.

This fixes an issue where the value of a display field changes after the
first creation and would not update it's displayed value.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 0.4.4

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

widget: add ConfirmDialog

this is a dialog that allows for confirming or canceling an action
easily.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

catalog_loader: don't try to load the catalog if the language is empty

this means that the Language value in the local storage is not set. by
default this would try to load `catalog-.mo` as language is an empty
string, so just skip trying to load something here and simply initialize
with an empty catalog instead, avoiding a failed load.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

widget: button: use 'fa-spin' for refresh icon

instead of loading icon, since the change is too jarring

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 0.4.3

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

buildsys: simplify upload target, add missing uses for BUILDDIR

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

focus: move code to dom module

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

align: move code to dom module

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

column: derive Properties instead of Default

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

card: derive Properties instead of Default

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

buildsys: add upload target

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: add DEBS and BUILDDIR variables

Make it a bit more complete and allow overriding the BUILDDIR one from
the outside.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

improve docs

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

improve size observer docs

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

row: derive Properties instead of Default

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

visibility observer: remove unused generic parameter, improve docs

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

panel: derive Properties instead of Default

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

bump version to 0.4.2

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

pwt-macros: bump version 0.3.0

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

add real VisibilityObserver component

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

rename VisibilityObserver to DomVisibilityObserver

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: use new SizeObserver component

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

add real SizeObserver component

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

rename SizeObserver to DomSizeObserver

And move it into the dom module.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

bump version to 0.4.1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

style fix line-length and formatting in various doc-comments

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

cleanup: remove unused code

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

widget: data table: beter handle navigation input

when inside a cell, we generally want to navigate with arrow keys,
except if we're inside an input element. In that case, use tab/shift+tab
to navigate out and don't catch arrows/space to allow navigation within
the input field (and input a space).

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

readme: fix future git repo url

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

reflow some doc-comments to use full 100cc width

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tree wide: remove debug log statements

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

check in generated d/control files

Makes it a bit easier to bootstrap and also see what changed w.r.t.
dependencies and meta information

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: wrap debcargo command over multiple lines

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

list: make number of prefetched rows configurable

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

theme density: rework naming to more telling variants

Rework the naming and doc comments of the theme density type but
include backward compatibility for the old variants when parsing the
density from a string. There's nothing to adapt on the actual UI
examples or products.

The bigger improvement is avoiding "Auto" for the default theme
spacing/density, as "auto" is widely used for something that
dynamically adapts on some environment aspects. But all our themes do
not have any such queries for high/low dpi or screen width to choose a
fitting spacing dynamically. Use `Preset` to avoid clashing with the
widely used `Default` trait from the rust std lib, albeit we could use
that if we really want.

The smaller fine tuning is renaming "High" to "Compact" and "Touch" to
"Relaxed".
The reasoning for the first is that most people, including our user
base, does not work with density a lot in their every day life, be it
physical density or screen density, so "higher density", while
definitively correct for what it does, is not that intuitive, i.e.
when "higher" makes things "smaller" we have a sort of double-negation
that is almost never good to have for UX.
The reasoning for the second (Touch -> Relaxed) is that this is not
doing anything with touch directly, it just might correlate with that
use case, and coupling a setting name with one specific use case is a
bit confusing for peoples coming from other use cases, like simply
having a higher DPI but not doing desktop wide scaling.

Note the naming was initially suggested by me with Thunderbird as
example, so this is really just improving on my own short comings from
the past. It also seems that newer Thunderbird Versions (e.g., latest
134.0b4) changed the wording to something similar than we now have
after this commit.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

pwt-macros: bump version to 0.2.0

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

fix debian release name

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

bump version to 0.4.0

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

widget: meter: add 'animated' flag

for adding the 'pwt-animated' class that animates the value transitions.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

list: also hide sparator behind wrapper

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: cleanup: avoid compiler warnings

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

css: add rust enum for font style

so we can write `FontStyle::DisplaySmall` instead of
`pwt-font-display-small` and get compile time checks.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

list: remove key property, because it is already in the standard widget properties

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: add separator property

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: split out list tile observer code

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: use list tiles with forcable height

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: remove separator property to simplify development

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list tile: add force_height property

This restricts the used container height, but still renders the tile with full
height so that we still get resize events with the correct size.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

tab: change default style to MaterialPrimary

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: add ARIA roles

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

cleanup: list: remove unused code, use better names

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: compute tail height

Instead of using a bad esitimation.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: fix for missing onscroll event.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: rate limit TileResize events

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: observe tile resizes

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list_tile: add resize_callback

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: improve inline docs

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

rename ProxmoxLanguageSelector to PwtLanguageSelector

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

docs: avoid useless re-export docs

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: improve docs

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

docs: fix compiler warning

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: improve docu

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

list: set scroll_to to correct size estimations

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>