This was a proposal for a browser rebrand that I posted on the Mozilla Wiki about a decade ago but I don't think many people saw it and too many were emotionally attached to the fox. Just also posting it here in case you wanted to use it. I knocked it up in MS Paint in about 10 minutes, so it's just for the concept and obviously not a polished final design or anything. Could change the colours if orange was too Firefox-like.

In T13888#245268, @SGOrava wrote:

It was also reported on bagtracker:
https://bugs.kde.org/show_bug.cgi?id=402835
https://bugs.kde.org/show_bug.cgi?id=429433

It was also reported on bagtracker:
https://bugs.kde.org/show_bug.cgi?id=402835
https://bugs.kde.org/show_bug.cgi?id=429433

Will submit an improved version on GitLab instead.

In D29007#663905, @carewolf wrote:

I for some reason can't land this change. It says "hook declined". Feel free to land it for me.

I for some reason can't land this change. It says "hook declined". Feel free to land it for me.

@drosca can you check this once.

keyword gets updated in the bookmark widget and organize bookmarks.

No idea, I'm not a Falkon developer.

In D28661#644983, @SGOrava wrote:

The other thing is:
This popup window can recognize if the site is already bookmarked or not.
In case it is already bookmarked it would be good to display keywords in keyword field.

fix the size of bookmark widget.

The other thing is:
This popup window can recognize if the site is already bookmarked or not.
In case it is already bookmarked it would be good to display keywords in keyword field.

Update Placeholder

Improve query filter

@drosca @dfaure @SGOrava
Do I need to add subscribers other than falkon? I have 3 other differentials that needs review but no response is made for some time, unless I mention by name.

Comment grammar correction

• I fixed "if"

Done requested change

Use qAsConst

Okay I fixed. thank you

QFile is closed in the destructor, this is not needed.

It should be <br/> instead.

https://forum.qt.io/topic/66191/new-line-n-doesn-t-work
As like this article, maybe dialog.setText(tr("...")) messages are translated in C type character type.
'\r\n' also would be works.
And checked with build source.

Used range loop as requested.

Using QString reference as requested.

Sorry ... It's my mistake .. I'll study hard more..

According to documentation it should be closed by Qt.
https://doc.qt.io/qt-5/qfile.html#dtor.QFile

The destructor of QFile will close it, doesn't it?

I tested it a little bit, and found out that the problem was not caused by 'name' attribute, but by 'key' - it is empty if an appropriate attribute is not present.
Now to check enter to https://my.amfoss.in/login (enter somethink and click save) and see there is not input
Then you can check it at any site (I choosed phabricator) - it should be autocompleted automatically

I think this is more better!
Because That causes an error. If you do that, you will not be able to automatically log in with the toolbar.

In "edit revision" in the sidebar in the right.

In D27791#630895, @ognarb wrote:

To improve the qualitity of the git history, it is recommanded to have a title that explain that it does (for example Add filter to Preferences > Extensions) and add BUG: 408312 in the summary so that the bug report is automatically closed when this diff is merged.

To improve the qualitity of the git history, it is recommanded to have a title that explain that it does (for example Add filter to Preferences > Extensions) and add BUG: 408312 in the summary so that the bug report is automatically closed when this diff is merged.

Fixed due to issues

@SGOrava can you please review this.

Stylistic fix

Done requested changes.

Popping from QUrl list instead of using an iterator, as requested.

Requested changes done.

Requested changes done.

bump? @drosca

bump? @drosca @SGOrava

Thank you for your contribution.

It's not supposed to create a new process, as falkon is designed to be a single process application. Nevertheless, I added a condition to send a new window command if an instance is running. This avoids process creation overhead.

There shouldn't be more than one Falkon process sharing one profile. The only exception is the private process, that is using (should be) all profile data in read-only mode.

In case you're wondering, Yes, I'm interested in becoming a GSoC student. I've contributed to different KDE projects including this year's SoK. However, this is my first patch to Falkon. Once this is done, you can recommend another bug to fix if you prefer.

@drosca sorry for the delay.

remove redundant code and add fill for email.

it looks if there is no name tag specified in the input tag, then we need to look for placeholder and I am filling the first part of the vector to username and the second to password.
anyways I will update the patch now.

I don't really like, there is ton of duplicated code. Also coding style is wrong.

@SGOrava @drosca is this good.