Re: [BUG] Re: The orgframe construct in the Beamer exporter as a default needs a rethink

[Pedro Andres Aranda Gutierrez]

To continue... my first reaction to the bug was to 

(setq org-beamer-frame-environment "frame")

to test. That resulted in

\newenvironment<>{frame}[1][]{\begin{frame}[environment=frame,#1]}{\end{frame}}

which is somehow weird and wrong.

This is why I propose to wrap the code in an

(unless (string= org-beamer-frame-environment "frame") ... )

so we are protected against people like me ;-)

Best, /PA 

On Sat, 2 Mar 2024 at 07:24, Pedro Andres Aranda Gutierrez <paaguti@gmail.com> wrote:

Hi Leo,

Wouldn't it be wiser to combine you fix with mine. I still think that setting org-beamer-frame-environment to "frame" when you don't need the fix and not emitting the extra newenvironment code in that case makes the generated LaTeX easier to read. Whether the default value should be frame org or orgframe is something we can debate.

Best, /PA

On Sat, 2 Mar 2024 at 00:12, Leo Butler <Leo.Butler@umanitoba.ca> wrote:

Hello,

Thanks for the bug report. The definition of the orgframe environment
did not pass the overlay spec to the underlying frame environment. I
believe the attached patch fixes that.

Attached is an org file that appears to exercise the patch and show that
it is doing the right thing. Could you confirm this, please?

----
Incidentally, I did propose that we introduce a BEAMER_FRAME property so
that it could be manually set, but Ihor did not like that idea so it got
scrapped.

Thanks,
Leo

--

Fragen sind nicht da, um beantwortet zu werden,

Fragen sind da um gestellt zu werden

Georg Kreisler

Headaches with a Juju log:

unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run a leader-deposed hook here, but we can't yet

--

Fragen sind nicht da, um beantwortet zu werden,

Fragen sind da um gestellt zu werden

Georg Kreisler

Headaches with a Juju log:

unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run a leader-deposed hook here, but we can't yet