- use key icon in connect/authenticate modal
- use person without hat for contributors
- use +- icon for diffs
- use color to distinguish between track/tracking states
We elminitate the element with the “button” role that wrapped the
Popover toggle slot. The element rendered into the slot is in all cases
already a button. This prevents issues where `getByRole("button")` finds
_two_ buttons for popover toggles.
We take the opportunity to simplify the popover toggling by making the
slot responsible for the toggle. This has a couple of advantages:
* We don’t need a `disabled` property for Popover. This is handled by
the button. (This fixes a bug we did not disable the PeerSelector
Popover properly.)
* The popover is opened on key events
In the future this also gives us more flexibility in determining how to
toggle the Popover.
Signed-off-by: Thomas Scholtes <geigerzaehler@axiom.fm>
Instead of sending a request for each label and assignee change on a
existing issue or patch, we should just do that for the new form,
where we wait for the confirmation for the final creation of the issue or patch.
Instead of showing the node id with a clipboard that copies a possible
wrong port number, this commit adds the externalAddresses config to the
node detail view, so node operators can define which externalAddresses
should show up.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
Co-authored-by: Rūdolfs Ošiņš <rudolfs@osins.org>
Also does some refactoring for `action`, `editable` and other variables
we were using which weren't very clear
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
We want the textarea to grow when a user writes longer text. This also
fixes the `resizable` property, which when `true` allows the user to
resize the textarea with their mouse.
Signed-off-by: Rūdolfs Ošiņš <rudolfs@osins.org>
This can have multiple reasons, interface has added a new property but
httpds haven't updated yet.
Other issues with some specific endpoints that return a 500 and we can't
load it.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
- This commit doesn't change the way syntax highlighting is done.
- But due to changes in `radicle-httpd` we can highlight the entire
files of a changeset, and then replace the plain text lines with
highlighted ones.
- Also highlights only the files currently visible of a changeset.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
libgit2 optimizes around not loading the content, when there's no
content callbacks configured.
So we aren't able to get a clear answer from radicle-surf if a file in a
diff is a binary or not.
A simple way without doing additional API requests is to check if the
file type is a executable and if there are no hunks (since we don't get
any deltas in binaries), this is a strong
indicator for a binary.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
Sebastian Martinez