Skip to content

Make unused settings in browser bookmark plugin functional or remove them #4536

Description

@DavidGBrett

These settings in the browser bookmark plugin are unused

public class Settings : BaseModel
{
    public bool OpenInNewBrowserWindow { get; set; } = true;

    public string BrowserPath { get; set; }

They don't appear in the UI and aren't considered when opening a bookmark

They should either be removed or made functional

BrowserPath is presumably intended to control which browser opens it - tbh its likely most people would want it to match the one the bookmark came from so maybe that would require a more complex solution instead of just making this exact setting functional as is. Such a feature is requested in #2805

The OpenInNewBrowserWindow setting should be easier to get working as we already support somethign similar in the url plugin

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions