On 28.03.22 15:07, Matthias Heiserer wrote: > The same code is used once in widget toolkit and twice in PVE already, > so it makes sense to add it as a separate button. > > Signed-off-by: Matthias Heiserer <m.heise...@proxmox.com> > --- > changes from v1: > move into separate class > rename vars to something a little bit nicer > move comment above function > reorder some statements > > Note: Because it's now guaranteed that the function is called at render time, > I think we can leave out the extra logic to test which text is currently set. > > > src/button/AltText.js | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > create mode 100644 src/button/AltText.js > >
applied with 2/3 squashed in, thanks! _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel