It is redundant, we can already go to the "Source"/"Files" tab by
clicking on the project title in the breadcrumbs.
This also fixes some of the links that were linking to "Source"
incorrectly.
- simplify positioning
- make sure they expand depending on content, yet have a min width
- add a popover for the connected button state, move httpd info from the
authenticated button into this popover
- fix external link styling
- make the hover outline color pink
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>
We still allow already published URLs with `/seeds/` in them to be
resolved.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
Signed-off-by: Rūdolfs Ošiņš <rudolfs@osins.org>
Instead of using always the `config.seeds.defaultHttpdPort` which is
443, we should take into account the `customPort` variable that is set by the
user.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
We switch to a project route type that is just a union. This has the
following advantages
* It’s much easier to construct project routes. Fewer properties are
needed and there are no nested objects.
* We eliminate the implicitly shared properties `path`, `peer`,
`revision` and `route` which only belonged to the tree route.
* The structure of the project route is more regular and thus easier to
understand.
Signed-off-by: Thomas Scholtes <geigerzaehler@axiom.fm>
We stop handling the URL hash through the routing system. Hashes are
local to the current view and can be handled by the shown components.
Signed-off-by: Thomas Scholtes <geigerzaehler@axiom.fm>
To make the code easier to reason about we handle all app URL encoding
and decoding and the related seed BaseUrl extraction in a single place.
Signed-off-by: Rūdolfs Ošiņš <rudolfs@osins.org>
We only supported fetching the default httpd port (8080).
This commit removes navigation to `radicle.local` which represents
`0.0.0.0:8080`, and prepares the app for defining custom port.
Signed-off-by: Sebastian Martinez <me@sebastinez.dev>
We set the logo size so that it takes up the same space before it is
loaded as when it is fully loaded. This way the search bar doesn't get
pulled to the left and there is no visual flickering.
Signed-off-by: Rūdolfs Ošiņš <rudolfs@osins.org>
Also don't show the connect button in the mobile layout. We don't
support running nodes on mobile anyway.
Signed-off-by: Rūdolfs Ošiņš <rudolfs@osins.org>
Rūdolfs Ošiņš