Opened 10 years ago
Closed 8 years ago
#5622 closed defect (bug) (wontfix)
Remove RTL CSS in favor of only generating RTL CSS as part of the new build tools
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | low | |
| Severity: | minor | Version: | |
| Component: | Core | Keywords: | |
| Cc: |
Description
via DJPaul ticket:5160#comment:49
5160-remove-rtl-css.diff
- The core team had (unfortunately, in private -- sorry) previously decided to keep RTL CSS files in trunk, mainly to support contributions to the project from people who expect RTL.
Thats fine, and would work fine similar to the imagemin and jshint tasks and have the compiled RTL CSS committed back to the /src folder.
- I think this change will be a
wontfix, but a new ticket would be the place to have a discussion about this if you feel strongly.
My opinion is indeed they are not needed, RTL is no longer an extra step, a second thought, it is now a feature of the build process and part of the workflow for any CSS change.
Anyway this can be easily implemented 'down the road', punt to Future Release or wontfix for use as a reference.
Change History (5)
#2
in reply to:
↑ 1
;
follow-up:
↓ 3
@
10 years ago
Replying to johnjamesjacoby:
I prefer the bbPress method of making CSS changes to
/srcand runninggrunt buildto test compiled changes
That doesn't seem directly related to this ticket, which is about removing RTL CSS files from trunk (and only generating them as part of a release step).
#3
in reply to:
↑ 2
@
10 years ago
Replying to DJPaul:
That doesn't seem directly related to this ticket, which is about removing RTL CSS files from trunk (and only generating them as part of a release step).
This sounds directly in conflict with what you said previously here:
The core team had (unfortunately, in private -- sorry) previously decided to keep RTL CSS files in trunk, mainly to support contributions to the project from people who expect RTL.
I prefer the bbPress method of making CSS changes to
/srcand runninggrunt buildto test compiled changes, including right-to-left changes. As it sits right now it's impossible to do this without running the release build runner, which seems incorrect.