Hi! I'm working on my first-ever TB patch, so be gentle. [is this the best place for this question?]
I hg'd the source, I build TB, and it runs (fine, as far as I can tell.) I went to start making changes to the code, but the file I wanted to edit is missing.
I found what I thought was the file to edit via dxr.mozilla.org. Specifically, it is comm-central/comm/mail/base/content/glodaFacetBindings.xml
I see that in the current tip at hg.mozilla.org there is no glodaFacetBindings.xml, so I guess that maybe it is no longer a current file (AFAICT it's last update was a couple years ago, so maybe it has been removed?)
That's fine, but I can't figure out where that functionality has gone. Specifically I'm looking for the code that handles the drop-down menu that chooses the sort-by for search results: "date" vs "relevance" (the div class is "results-message-sort-bar").
For example, here is the glodaFaceBindings.xml line that handles when "relevance" is chosen. No matter how I search, I can't seem to dig up the new location of that functionality. (I've been searching using dxr.mozilla.org, as well as searching through my local source tree.) I've been using things like "results-message-sort-bar" and "sortBy" and "FacetContext.sortBy" as search terms, as they are the relevant (and current) hooks, AFAIK, that the missing functionality ties in to.
Any help? Thanks!
TB: missing source file? How to trace the functionality?
-
- Posts: 43
- Joined: October 28th, 2006, 11:17 pm
-
- Posts: 1361
- Joined: December 15th, 2015, 1:20 pm
Re: TB: missing source file? How to trace the functionality?
dxr is on mozillas "do not care" list. comm-central is broken for ages now. I usually use comm-release which seems to be updated now and then. The file you try to edit is probably an xbl binding. This has been completely removed by mozilla and the TB devs after esr 60.
Searchfox is supposed to be the greatest replacemnnt of all time for dxr but the interface lacks and the blame function is attrocious:
https://searchfox.org/comm-central/source
But at least it is up to date.
TB development discussions moved from moznet maildev to matrix. Closed doors but getting an account to access it should be easy. Only SeaMonkey stayed on irc freenet.
FRG
Searchfox is supposed to be the greatest replacemnnt of all time for dxr but the interface lacks and the blame function is attrocious:
https://searchfox.org/comm-central/source
But at least it is up to date.
TB development discussions moved from moznet maildev to matrix. Closed doors but getting an account to access it should be easy. Only SeaMonkey stayed on irc freenet.
FRG
-
- Posts: 1361
- Joined: December 15th, 2015, 1:20 pm
Re: TB: missing source file? How to trace the functionality?
Checking the file in hg history it seems was removed in :
Bug 1517523 - Migrate results-message binding to custom element. r=mkmelin
Functionality is now supposed to be in mail/base/content/glodaFacet.js
Bug 1517523 - Migrate results-message binding to custom element. r=mkmelin
Functionality is now supposed to be in mail/base/content/glodaFacet.js
-
- Posts: 43
- Joined: October 28th, 2006, 11:17 pm
Re: TB: missing source file? How to trace the functionality?
Thanks! I will explore further!
To be clear, when you say "comm-central is broken for ages now" you mean on dxr, right? comm-central is still the proper hg repo for the tip of TB development, right?
To be clear, when you say "comm-central is broken for ages now" you mean on dxr, right? comm-central is still the proper hg repo for the tip of TB development, right?
-
- Posts: 43
- Joined: October 28th, 2006, 11:17 pm
Re: TB: missing source file? How to trace the functionality?
Also: patch submitted; hopefully I didn't mess it up too much: https://phabricator.services.mozilla.com/D74731