Forking customize-titlebar: Title Bar Customizer

Dedicated board for extension releases/support threads

Moderators: FranklinDM, Lootyhoof

Forum rules
Please do not create new topics here unless you are an extension author in need of a dedicated releases&support thread!
User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-08, 00:15

Since there is an announcement about soon unsupported Firefox legacy extensions and gathering about info for such extensions; I tried to fork one of my favourite extension, customize_titlebar_v2, according to forking tips posted here. I'm not a web developer and don't know these stuff but I think this fork would work. I changed extension license from MPL-GPL-LGPL trio license to GPLv2. Anyway please inspect it and test. I hope it works for you too.

Source code and relevant extension xpi can be found here : https://github.com/tarakbumba/title-bar ... -palemoon/

User avatar
moonbat
Knows the dark side
Knows the dark side
Posts: 4942
Joined: 2015-12-09, 15:45
Contact:

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by moonbat » 2021-04-08, 00:42

tarakbumba wrote:
2021-04-08, 00:15
I changed extension license from MPL-GPL-LGPL trio license to GPLv2
Better check if you're allowed to do that.
"One hosts to look them up, one DNS to find them and in the darkness BIND them."

Image
Linux Mint 21 Xfce x64 on HP i5-5200 laptop, 12 GB RAM.
AutoPageColor|PermissionsPlus|PMPlayer|Pure URL|RecordRewind|TextFX

New Tobin Paradigm

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by New Tobin Paradigm » 2021-04-08, 01:00

Remove META-INF completely that is jar signing and Mozilla jar signing that won't be valid.

Your install manifest is badly formatted and using an obsolete incarnation of the attr form (though still works it is very sub-optimal). It also wouldn't pass Phoebus Validation anyway. Let me rewrite it for you:

Code: Select all

<?xml version="1.0"?>

<RDF xmlns="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
     xmlns:em="http://www.mozilla.org/2004/em-rdf#">

  <Description about="urn:mozilla:install-manifest"
               em:type="2"
               em:id="title-bar-customizer@github.org"
               em:version="1.0"
               em:name="Title Bar Customizer"
               em:description="Customize the browser titlebar"
               em:creator="tarakbumba"
               em:homepageURL="https://github.com/tarakbumba/title-bar-customizer-for-palemoon"
               em:iconURL="chrome://titlebar/titlebar-logo.png"
               em:optionsType="1"
               em:optionsURL="chrome://titlebar/content/options.xul">
    <em:contributor>Richard Berry titlebar@sidthemonkey.com</em:contributor>
    <em:contributor>Ivy Durante</em:contributor>
    <em:contributor>Chris Neuner http://unrealstuff.bplaced.de</em:contributor>
    <em:contributor>Leszek(teo)Zyczkowski</em:contributor>
    <em:contributor>el_libre www.catmidia.cat</em:contributor>
    <em:contributor>hrvoje.solc at gmail.com</em:contributor>
    <em:targetApplication name="Pale Moon">
      <Description em:id="{8de7fcbb-c55c-4fbe-bfc5-fc555c87dbc4}"
                   em:minVersion="28.0.0a1"
                   em:maxVersion="29.*"/>
    </em:targetApplication>
  </Description>
</RDF>
I would also seriously recommend you go through set your text editor to use spaces for indentation and convert all the tabs to spaces. I recommend 2 spaces per indent level. It will not only make it easier to deal with for your self but for others to contribute to.

As for the licensing you can fork as GPL2 but I would resist going further to GPL3. But yes, Moonbat the Tri-license does allow that. I think you should just keep it as MPL 1.1 or upgrade it to MPL 2.0. It is more sane and less prone to licensing conflicts and resistant to further hostile assimilative activity.

On the subject of function testing, I will leave that to other community peers.

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-08, 12:00

Thank you very much Tobin. I'll add your rewritten install.rdf asap and do other modifications. Regarding license, İ think i'll keep gpl2 for now.

EDIT: Title Bar Customizer v. 1.1 released including modifications provided by Tobin.

New Tobin Paradigm

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by New Tobin Paradigm » 2021-04-08, 12:38

You gonna submit it to the Add-ons Site?

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-08, 13:26

New Tobin Paradigm wrote:
2021-04-08, 12:38
You gonna submit it to the Add-ons Site?
I want to but don't know how. As I said I have almost no knowledge about these extensions but just wanted to save this particular one.

New Tobin Paradigm

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by New Tobin Paradigm » 2021-04-08, 13:33

You'll learn over time, I'm sure. You are already suceeding by even bothering to go this far.

Uhh, here: http://developer.palemoon.org/addons/site/

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-09, 20:54

Updated to version 1.3:
  • Added auto update capatibilty from github.
  • Changed extension's unique id to a uuid

vannilla
Moon Magic practitioner
Moon Magic practitioner
Posts: 2181
Joined: 2018-05-05, 13:29

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by vannilla » 2021-04-09, 21:02

tarakbumba wrote:
2021-04-09, 20:54
Added auto update capatibilty from github.
Wouldn't this be solved by the add-ons website?
Or do you plan to keep this as external?

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-09, 23:17

vannilla wrote:
2021-04-09, 21:02
tarakbumba wrote:
2021-04-09, 20:54
Added auto update capatibilty from github.
Wouldn't this be solved by the add-ons website?
Or do you plan to keep this as external?
I can't register to addons site. Phoebus just doesn't send verification email. So, I decided to add this capability for now as external until problem solved.

New Tobin Paradigm

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by New Tobin Paradigm » 2021-04-09, 23:56

I skipped Thursday except for a few hours to eat something because Monday through Wednesday I only had maybe 10 hours total sleep. You shouldn't be contacting me for issues with registration but the Add-ons Team.

According to mail log, Google bounced it back. Easy fix tho.. All one of us has to do is go into your account and hit submit which will put it in validated status.

AND... You're all set. You will have release a version without an em:updateURL to your impromptu update.rdf so they get switched over.

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35402
Joined: 2011-08-28, 17:27
Location: Motala, SE
Contact:

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by Moonchild » 2021-04-10, 08:38

FTR the mail issue should also be fixed now. looks like mail configuration got lost resulting in the server trying to send mail directly to Google which for palemoon.org mail isn't a good idea because SPF etc.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

New Tobin Paradigm

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by New Tobin Paradigm » 2021-04-10, 09:18

Still may end up unfairly in spam but least it SHOULD get there now.

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-10, 10:34

New Tobin Paradigm wrote:
2021-04-09, 23:56
I skipped Thursday except for a few hours to eat something because Monday through Wednesday I only had maybe 10 hours total sleep. You shouldn't be contacting me for issues with registration but the Add-ons Team.

According to mail log, Google bounced it back. Easy fix tho.. All one of us has to do is go into your account and hit submit which will put it in validated status.

AND... You're all set. You will have release a version without an em:updateURL to your impromptu update.rdf so they get switched over.
Thank you very much. Will I release this new update at my github? Or at addons site?
Last edited by tarakbumba on 2021-04-10, 10:42, edited 2 times in total.

New Tobin Paradigm

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by New Tobin Paradigm » 2021-04-10, 10:39

You should update on github and your update.rdf file but don't have em:updateURL in there and just submit on the Add-ons Site.. afterword you only need to update on the Add-ons Site but of course you can still release on github too just don't need to care about update.rdf.

It is all pretty easy tho. Designed it to be.

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-10, 11:09

New Tobin Paradigm wrote:
2021-04-10, 10:39
You should update on github and your update.rdf file but don't have em:updateURL in there and just submit on the Add-ons Site.. afterword you only need to update on the Add-ons Site but of course you can still release on github too just don't need to care about update.rdf.

It is all pretty easy tho. Designed it to be.
Thank you. I updated extension to 1.4 version with removal of update url and after that submitted to Add-ons Site.

User avatar
back2themoon
Moon Magic practitioner
Moon Magic practitioner
Posts: 2358
Joined: 2012-08-19, 20:32

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by back2themoon » 2021-04-11, 09:31

Many thanks and well done for releasing it. :thumbup:

User avatar
tarakbumba
Moonbather
Moonbather
Posts: 54
Joined: 2021-01-18, 14:26

Re: Forking customize-titlebar: Title Bar Customizer

Unread post by tarakbumba » 2021-04-11, 12:42

back2themoon wrote:
2021-04-11, 09:31
Many thanks and well done for releasing it. :thumbup:
Thank you. :thumbup:

Locked