Am Dienstag, den 18.08.2020, 22:06 +0200 schrieb Pavel Sanda: > On Tue, Aug 18, 2020 at 09:23:08AM +0200, Jürgen Spitzmüller wrote: > > > Item "Settings...|S" "inset-settings" > > > + Item "Open citation content|O" "inset-edit" > > > > This should be an OptItem, and the function should be disabled if > > the > > citation in question does not have a target to follow. > > Not easy, you don't know the result of external script. Calling it > by default and implementing return values looks like going too far.
I meant if getLocators() return nothing for all fields (if the citation[s] in question neither has url nor another field which is considered). > > Also, we use Title Casing in menus. > > Fixed. Thanks. > > > + doi = operator[]("doi"); > > > + if (!doi.empty() && !prefixIs(doi,from_ascii("http"))) > > > + doi = "https://doi.org/" + doi; > > > + url = operator[]("url"); > > > + file = operator[]("file"); > > > > Biblatex also has eprint (a template identifier that can point to > > arxiv, jstor etc., which is indicated by the eprinttype field). A > > bit > > harder to resolve, but possible at least for the default > > eprinttypes as > > preset by Biblatex). See Biblatex manual, 3.13.7. > > I implemented all types listed in the manual. Does it work for you? Yes. Thanks. > > > > + //TODO I have no clue about non-bibtex cases > > > > This is the bibliography environment. All you could do here is let > > LyX > > scan the entry for URL or HRef insets. But I'm not sure it's worth > > it. > > I added your comment ATM. If someone asks we can add it. Fair enough. > > > + > > > +void showTarget(string const & target){ > > > + lyxerr<<"Showtarget:" << target << "\n"; > > > > You will want to remove this debug msg, or make it a proper dbg > > message. > > Sure, oversight. Thanks, Jürgen > > Pavel
signature.asc
Description: This is a digitally signed message part
-- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel