I suspect that 'ExpiresByType text/css "access plus 50 minutes"' from eg.conf is the issue, because CSS normally doesn't change very often. That cache time should be more like 500 days, but we need to merge the cache-busting bug # 1681095 first.
For this branch, we could just append "?dir=rtl" or "?dir=ltr" to the style.css URL in src/templates/opac/parts/base.tt2
So the base.tt2 line would become something like:
<link rel="stylesheet" type="text/css" href="[% ctx.opac_root %]/css/style.css?dir=[%
IF ctx.get_i18n_l(ctx.eg_locale).rtl == 't'
%]rtl[%
ELSE
%]ltr[% END %]"
Then when the cache-busting branch is merged, we append the extra cache-busting GET param accordingly.
I suspect that 'ExpiresByType text/css "access plus 50 minutes"' from eg.conf is the issue, because CSS normally doesn't change very often. That cache time should be more like 500 days, but we need to merge the cache-busting bug # 1681095 first.
For this branch, we could just append "?dir=rtl" or "?dir=ltr" to the style.css URL in src/templates/ opac/parts/ base.tt2
So the base.tt2 line would become something like:
<link rel="stylesheet" type="text/css" href="[% ctx.opac_root %]/css/ style.css? dir=[% i18n_l( ctx.eg_ locale) .rtl == 't'
IF ctx.get_
%]rtl[%
ELSE
%]ltr[% END %]"
Then when the cache-busting branch is merged, we append the extra cache-busting GET param accordingly.