ladybird icon indicating copy to clipboard operation
ladybird copied to clipboard

Added some settings options and a new menu item

Open yeppiidev opened this issue 3 years ago • 5 comments

Added the following:

  • Always show tabs setting
  • Default new tab page setting
  • Close all tabs except the current one menu item (in File menu)

yeppiidev avatar Aug 14 '22 06:08 yeppiidev

@Xexxa it would be nice if serenityos.net had a new tab search page

yeppiidev avatar Aug 14 '22 07:08 yeppiidev

Added a status bar, an 'always show tabs' option which will make the TabView always show tabs even if there's only a single tab, and a close all but current tab option

yeppiidev avatar Aug 14 '22 09:08 yeppiidev

Hi @yeppiidev. I obviously feel honored that you added links to serenityos.net and wiki.serenityos.net, I hope you noticed that serenityos.org is the official site.

I added at your request https://ladybird.serenityos.net/new-tab.html but it would be better to use the file in the SerenityOS repo https://github.com/SerenityOS/serenity/blob/master/Base/res/html/misc/new-tab.html or if that is not possible, host a copy in the Ladybird repo.

I don't think a help menu would be merged after Andreas earlier statement: "But for now, please treat ladybird as a debugging tool for LibWeb. :)"

The same goes with the statusbar, it was removed in an earlier PR

I would assume the additional settings could be merged.

Personally I like statusbars and help menus so I downstream many rejected PRs to my fork :^) Just giving some useful info if you want this PR to get merged ;-)

Xexxa avatar Aug 14 '22 17:08 Xexxa

Thanks for the tips! I will remove the help menu (I too thought it was useless) and the statusbar 👍🏼

yeppiidev avatar Aug 15 '22 11:08 yeppiidev

Oh, I have no idea how to load files from the filesystem so I'm leaving that one for someone else

yeppiidev avatar Aug 15 '22 11:08 yeppiidev

This has conflicts.

AtkinsSJ avatar Sep 16 '22 17:09 AtkinsSJ

IMHO - this is safe to close, as main drifted from this branch way too much

diegoiast avatar Nov 13 '22 19:11 diegoiast

Right, this has tons of conflicts and is almost 3 months at this point. if you rebase it and rework it I'm sure we could revive the spirit of the changes, which seem overall like a good idea. thanks for your contributions!

ADKaster avatar Nov 13 '22 20:11 ADKaster

yeah, i kind of messed up this PR so it was better to close this anyway. will contribute to serenity when possible 🙂

yeppiidev avatar Nov 19 '22 07:11 yeppiidev