Re: Removing "MathML View Source" menu item? [was: Can we remove nsIEntityConverter?]

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: Removing "MathML View Source" menu item? [was: Can we remove nsIEntityConverter?]

J. Ryan Stinnett
As an occasional contributor to view source, I think it would be nice to
remove the MathML support from m-c like you are proposing, as it definitely
increased the maintenance burden when we reworked view source into a tab a
while ago, and my assumption is the usage of MathML view source is
relatively low (though I currently have no data to back that up).

Looking at the add-on, it seems to miss one feature: Currently in m-c, you
can select a portion of a MathML expression, choose "View Selection
Source", and see the MathML source for the selection. If that's added to
your add-on, I believe you'd have feature parity with the existing MathML
view source support.

I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1365626 to remove
MathML view source from m-c.

- Ryan

On Sat, May 6, 2017 at 5:40 AM, Frédéric Wang <[hidden email]> wrote:

> Le 30/04/2016 à 12:25, Henri Sivonen a écrit :
> > We ship data tables for converting from Unicode to HTML entities.
> > These tables obviously take space. (They are not optimized for space
> > usage, either.) As far as I can tell, these tables are not used at all
> > in Fennec. In desktop Firefox, these data tables are used only for the
> > MathML View Source feature.
> >
> > Additionally, a subset of the tables is used by some XPCOM-based
> > extensions, but those extensions seem to be obsolete or abandoned or
> > don't seem to be using the feature for a very good reason.
> >
> > These data tables are not exposed to the Web Platform.
> >
> > In https://bugzilla.mozilla.org/show_bug.cgi?id=1048191 I proposed
> > removing this for mobile only, but how about we just remove this
> > altogether in order to make both Fennec and desktop Firefox smaller?
> >
>
> Hi,
>
> I'm resurrecting this old thread, just to say that there is now a
> WebExtension implementing the "MathML view source" feature (using better
> syntax highlighting and handling the invisible spaces too):
>
> https://addons.mozilla.org/en-US/firefox/addon/mathml-view-source/
>
> So I'm proposing to remove that feature from the core mozilla source.
> The only concern I have is for Thunderbird/Seamonkey as it is not
> clear yet what will be the future regarding WebExtensions.
>
> Any opinions?
>
> --
> Frédéric Wang
> _______________________________________________
> dev-platform mailing list
> [hidden email]
> https://lists.mozilla.org/listinfo/dev-platform
>
_______________________________________________
dev-apps-thunderbird mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-apps-thunderbird
Reply | Threaded
Open this post in threaded view
|

Re: Removing "MathML View Source" menu item? [was: Can we remove nsIEntityConverter?]

Frédéric WANG
Le 17/05/2017 à 17:14, J. Ryan Stinnett a écrit :
> Looking at the add-on, it seems to miss one feature: Currently in m-c, you
> can select a portion of a MathML expression, choose "View Selection
> Source", and see the MathML source for the selection. If that's added to
> your add-on, I believe you'd have feature parity with the existing MathML
> view source support.

Right, I had opened
https://github.com/fred-wang/webextension-mathml-view-source/issues/7
for missing features. I don't know exactly how important they are, though.

> I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1365626 to remove
> MathML view source from m-c.

Thanks, maybe a first step would be to remove the usage of
nsIEntityConverter as Henri indicated as I don't think it's an important
feature (this is
https://github.com/fred-wang/webextension-mathml-view-source/issues/6).


--
Frédéric Wang
_______________________________________________
dev-apps-thunderbird mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-apps-thunderbird