Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
As can be seen in http://drupal.org/files/issues/loggin.jpg (from #706862: Add RTL styling to Bartik)
Comment | File | Size | Author |
---|---|---|---|
#123 | shortcut.png | 558 bytes | aspilicious |
#117 | toolbar.png | 29.39 KB | aspilicious |
#117 | toolbar-rtl-final-fix.patch | 4.56 KB | aspilicious |
#112 | ie-toolbar.png | 197.43 KB | Jeff Burnz |
#105 | toolbar-shortcuts-rtl_740182.patch | 5.14 KB | Jeff Burnz |
Comments
Comment #1
Damien Tournoud CreditAttribution: Damien Tournoud commentedComment #2
yoroy CreditAttribution: yoroy commentedYes, that's a better title, thanks. First draft of a patch
Comment #4
Bojhan CreditAttribution: Bojhan commented#2: rtl-toolbar-shortcuts.patch queued for re-testing.
Comment #6
aspilicious CreditAttribution: aspilicious commented#2: rtl-toolbar-shortcuts.patch queued for re-testing.
Comment #7
JacineJust tried to test this, but the patch no longer applies.
Comment #8
aspilicious CreditAttribution: aspilicious commentedA reroll
Comment #9
bleen CreditAttribution: bleen commentedmissing space
Powered by Dreditor.
Comment #10
yoroy CreditAttribution: yoroy commentedFixed that and another similar missing space
Comment #11
cosmicdreams CreditAttribution: cosmicdreams commentedcool, I'll give this one a test.
Comment #12
cosmicdreams CreditAttribution: cosmicdreams commentedI'm testing this right now. I just wanted to report that I didn't experience the issue reported by the OP when I started from a fresh cvs install of d7 and switched english to right to left through the Locale module. I'll see if this clears up some the issues I am seeing though (these issues are not related to the toolbar).
Comment #13
cosmicdreams CreditAttribution: cosmicdreams commentedok here are some results : (see images)
as the pictures show there are still some issues with the order of the menu items and the overlapping of ui elements and words. I am suspicious of the accuracy of my test though, since there seems to be no change from the cvs HEAD after applying the patch.
Tested with Chrome 5 beta and Firefox 3.5
Testing method: compare and contract the ui before and after the patch, flushed cache. With Firefox I disabled browser-side caching.
Comment #14
cosmicdreams CreditAttribution: cosmicdreams commentedSo I guess this brings up questions about what we intend the page to look like after the RLT styling is full implemented. What should a perfect RTL toolbar look like?
Comment #15
aspilicious CreditAttribution: aspilicious commentedI made a new patch, fixing the toolbar.
Needs review :)
Comment #16
aspilicious CreditAttribution: aspilicious commentedDeleted original comment, need more thinking time...
Edit: I need to reroll this cause we need to reverse the order of the menu items
Comment #17
aspilicious CreditAttribution: aspilicious commentedReroll
Comment #18
aspilicious CreditAttribution: aspilicious commentedThis is not done yet...
The add shortcut button, need a proper rtl styling...
You need to add these lines to shortcut.css rtl version to reverse the rounded border:
We also need to add rtl style images to shortcut.png (in rtl we always see the hover box starting on the right of the add button)
Comment #19
aspilicious CreditAttribution: aspilicious commentedAnd we are learning a lot about patching ;).
This one works great!
Still no support for IE7 or below, but hey it's a start...
You need to replace the shortcut image with the new one I made.
You need the overlay rtl patch if you want some nice results ;)
Opening that issue soon, and I also will refer to the contextual rtl patch!
Seven needs rtl styling to, thats for another patch...
Comment #20
aspilicious CreditAttribution: aspilicious commentedthis patch in combination with this one #766170: Overlay lacks rtl styling gives the result seen in the screenshot :)!
Comment #21
aspilicious CreditAttribution: aspilicious commentedCrosslinking:
#765856: Contextual lack rtl styling
#766624: Dashboard lacks rtl styling
#766458: Seven theme lacks rtl styling
#766170: Overlay lacks rtl styling
These patches don't work in IE7 and below!
Comment #22
aspilicious CreditAttribution: aspilicious commentedSee previous post...
Comment #23
sunOverridden in RTL, but does not use /* LTR */ pointer.
Trailing white-space here.
Should be on separate lines.
This should not happen.
Powered by Dreditor.
Comment #24
aspilicious CreditAttribution: aspilicious commentedOk another try...
Comment #25
aspilicious CreditAttribution: aspilicious commentedOk I fixed IE7 (a stupid float that couldn't be overridden)...
To summarize what is done for lazy people, or people with not much time.
This patch gives full rtl support for the toolbar and shirtcut module.
It is tested in:
IE7
IE8
IE9 (technical preview)
Chrome
Firefox 3.0
Firefox 3.6
Opera 10.5
Safari
Can't test in IE6.
This patch needs the update shortcut.png that you can find in #19...
If you wonna test this please install the patches listed in #21, without overlay rtl patch / seven rtl patch this one is hard to test
FYI: if you install IE9 technical preview you can have developer tools for most ie versions (IE7, IE8 and IE9), thats how I found the IE7 problem
Comment #26
james.elliott CreditAttribution: james.elliott commentedSubscribing
Comment #27
aspilicious CreditAttribution: aspilicious commentedI installed IE6 in my brand new xpmode virtual machine.
It looks bad but it "works" as good as the non rtl version, screenshot attached.
Comment #28
aspilicious CreditAttribution: aspilicious commentedThis adds a lil IE6 tweak
Comment #29
realityloopHorizontal scroll with goes crazy in certain areas of the site, see screenshot taken in FireFox 3.6.3
http://img.skitch.com/20100421-t1jnw3byue7i51q4en58m95ny5.png
Comment #30
yoroy CreditAttribution: yoroy commentedAny hint on what is causing this?
Comment #31
aspilicious CreditAttribution: aspilicious commentedI guess some position:relative issues or something like that
Comment #32
realityloopI've been able to ascertian that the skip link is causing it, setting it to display: none fixes layout on page, but obviously not the way to deal with it.
Comment #33
realityloopThis should fix horizontal scrolling issue from #29
Comment #34
Elijah LynnTested. Looks good.
Comment #35
tsi CreditAttribution: tsi commentedTested on IE6+, FF 3.x, Chromium.
IE6 is not perfect obviously (rounded corners, transparent PNGs) but who cares...
Looks good.
Comment #36
yoroy CreditAttribution: yoroy commentedCool, thanks all. rtbc indeed.
Comment #37
aspilicious CreditAttribution: aspilicious commented#33 I'm curious what you changed...
Comment #38
realityloop#37 it's in the patch.. added following to modules/shortcut/shortcut-rtl.css
+div#skip-link {
+ margin-left: 500000px;
+}
Comment #39
realityloopoops.. I accidentally changed status in previous reply
Comment #40
aspilicious CreditAttribution: aspilicious commentedseems like a little hack, can't we fix this with something nicer?
Comment #41
yoroy CreditAttribution: yoroy commentedEw! :) yeah could do with a nicer solution.
Comment #42
realityloopAny suggestions? It doesn't need to be that many pixels if thats the issue.. it's just a random number I chose..
Comment #43
sunI can't help, because I cannot review the actual changes. Please remove those additional CSS clean-ups - they have nothing to do with this critical issue.
Comment #44
aspilicious CreditAttribution: aspilicious commentedCome on sun...
1) I just moved some css, so that the order is alphabeticly
I didn't change one line!
2) Look at the rtl version if you want to review...
This makes me sad :(
3) It's not a huge patch so can you please look over the small cleanup?
And if this is a critical issue the other rtl patches are critical too
Comment #45
yoroy CreditAttribution: yoroy commentedWell, sun may be strict here, but he's also right. You really have to stick to the actual issue at hand. Sneaking in other changes disrupts other peoples workflow, no matter how harmless they are.
Comment #46
aspilicious CreditAttribution: aspilicious commentedArgh :(...
I started with patching this thing 2 weeks ago, if at least one of you can promise me to review this today I'll clean this patch. ;)
(I need the promise to overcome the sadness ;) )
Comment #47
realityloopI don't quite understand how this is a hack, given that everything has been switched rtl, it seems to make sense to me that the margin for taking the skip link off screen would also have to switch to the other side as well..?
Does anyone have time to enligten me, I'd like to understand.
Comment #48
aspilicious CreditAttribution: aspilicious commentedDo we have the margin in the normal css to?
Oow...
Wait... I remeber it I think...
Well I'll clean this patch and reroll it and hopefully sun or jacine will give their opinion.
Comment #49
realityloopYes, the default style.css has the following:
I tried a negative right-margin but it didn't work, hence my solution.
This will still be accesible to screen readers which I think is the primary issue..?
Comment #50
aspilicious CreditAttribution: aspilicious commentedThe negative margin is probably to hide the text.
Comment #51
realityloop#50, which is what my large positive margin is for on rtl version :)
Comment #52
sun#skip-link doesn't belong to Shortcut module.
Also, where is the LTR style for this? (it needs the /* LTR */ pointer)
Lastly, this margin will result in a horizontal scrollbar.
Broken selector/code here.
Powered by Dreditor.
Comment #53
aspilicious CreditAttribution: aspilicious commentedHmm can you try to find the lowest negative value we can use?
Comment #54
realityloop#52 It doesn't result in a horizontal scrollbar, it has been tested in multiple browsers (#35), though I agree that it doesn't belong in Shortcut module.
I will split it out and add the pointer at the code sprint in San Francisco today.
Comment #55
aspilicious CreditAttribution: aspilicious commentedHere is a cleaned version of the patch (I hope I didn't miss anything)
Comment #56
aspilicious CreditAttribution: aspilicious commented#52 damnit, thats why we need seven RTL styling!!!!!
That is the place to put that in...
And I fixed the other thingie sun is mentioning...
I'll remove the skip link.
Comment #57
aspilicious CreditAttribution: aspilicious commentedReroll see #56 and #52
Comment #58
sunDon't see a LTR pointer for this in this patch?
Trailing white-space here. "px" can be omitted.
(and possibly elsewhere) I believe the margin-left additionally needs to be reset to zero.
94 critical left. Go review some!
Comment #59
aspilicious CreditAttribution: aspilicious commentedOk fixed everything, thnx sun!
This needs screenshot reviews now.
I'll also test it.
Comment #60
realityloopSo the Skip link text fix not present now?
Screenshot shows result of v9 patch, possibly still needs work.
http://img.skitch.com/20100422-rark2jepqqu181yfg5i94t2jwu.png
This may have been cache issue at my end, it's displaying ok now.
Hopefully someone else can check and mark RTBC
Comment #61
aspilicious CreditAttribution: aspilicious commentedInstall the other rtl patches (see #21) to test this one, cause seven theme, overlay, ... doesn't have rtl at the moment...
After installing those I can't see any rtl error (caused by shortcut or toolbar).
The skip link is part of seven-rtl.css (other issue)
So please install those patches first, else this one never gets commited...
EDIT: PatchV10 fixes a small issue with add shortcut hover links
Comment #62
realityloop#61 feel free to test my patch for "Seven" :)
http://drupal.org/node/778964
Comment #63
aspilicious CreditAttribution: aspilicious commentedYour seven patch is far from complete ;). I marked it as duplicate.
Work on #766458: Seven theme lacks rtl styling if you wonna help.
Comment #64
tsi CreditAttribution: tsi commentedTested last patch on IE7, IE8 (with IETester) FF3.x, Chrome, safari4win on winXP
FF3.x and Chromium on Ubuntu
* Probably nothing - No gray background on hover in IE8 via IETester but that might be related to IETester.
* Menu elements are floated to the left in RTL, that means that they don't flip their order, in RTL - last menu item should be on the left, but I couldn't fix this without breaking IE7.
Comment #65
aspilicious CreditAttribution: aspilicious commentedTSI
1) please show screenshots. :)
2) "menu items" is that has to do something with the overlay/shortcuts?
Comment #66
tsi CreditAttribution: tsi commented1) I can create screenshots but there's nothing to show really, it looks the same as your last one in #61
2) I'm not sure I understand the question, but I don't think it does, my point is that "content" - the first menu item (in the toolbar) - should be placed on the right of "structure" - the menu item that comes right after him and so on...
Comment #67
aspilicious CreditAttribution: aspilicious commentedOw... I had a patch with that change... who deleted that part :s
Comment #68
aspilicious CreditAttribution: aspilicious commentedGonna figure it out
EDIT: hmm I deleted it to fix IE7, but that wasn't the right way... Need help with it...
EDIT2:
you need to add in toolbar-rtl.css if you wonna help us figure this out (that line doesn't work in IE7)
Comment #69
tsi CreditAttribution: tsi commentedTried that already, that's why I wrote "but I couldn't fix this without breaking IE7"
Comment #70
Jeff Burnz CreditAttribution: Jeff Burnz commented#61: rtl-toolbar-shortcutsV10.patch queued for re-testing.
Comment #72
aspilicious CreditAttribution: aspilicious commentedBumping...
Someone with a soluton for the IE problem?
Comment #73
casey CreditAttribution: casey commentedFloating elements to the right inside a liquid left floated element seems to be impossible in IE6/IE7; giving the ULs a width makes the list items show up again, but we can't set no width as the number of list items is not fixed.
Therefox I suggest a CSS hack for IE6/IE7 that uses float:left on list items.
Comment #74
aspilicious CreditAttribution: aspilicious commentedCasey you just disabled rtl for that part of the toolbar. It doesn't look ugly anymore but it isn't really a fix...
Comment #75
aspilicious CreditAttribution: aspilicious commentedAnd her eis a discusing about the problem: http://www.sitepoint.com/forums/showthread.php?t=624584
Comment #76
JohnAlbinSubscribe.
Comment #77
catchValid bug but does not block release.
Comment #78
aspilicious CreditAttribution: aspilicious commentedIt does block release as rtl styling is a must for core themes...
We *need* more effort on this....
Comment #79
catchAn admin-only-rtl-language-only-optional-module-only bug does not meet the definition of critical. That doesn't mean it can't be fixed before release, but we need perspective on what critical issues actually are.
Comment #80
Jeff Burnz CreditAttribution: Jeff Burnz commented#73: toolbar+shortcut-rtl11.patch queued for re-testing.
Comment #82
tsi CreditAttribution: tsi commentedThe last patch applied for me only after deleting the first lines :
anyway I thought there must be a better way than putting IE hacks in core (how many kittens die when this happens ?), then I remembered that IE really hates floating a elements so I gave them float:none and fixed some issues caused by that move and here is the result, it is not perfect but I guess this is the best we can do with the current structure of the toolbar.
Comment #83
tsi CreditAttribution: tsi commentedNeeds review...
Comment #84
tsi CreditAttribution: tsi commentedSpeechless ?
I'll start - the week point of this patch is that it is not a perfect mirror of the ltr style, but in this case a perfect mirror just won't do the trick.
Comment #85
bleen CreditAttribution: bleen commentedRE #84 ... I dont think that is a weakness at all. The only factors that should affect the RTL styling of the toolbar are what looks best RTL and what is the best UX for RTL users. The LTR experience should have nothing to do with it.
Comment #86
tsi CreditAttribution: tsi commented@bleen18 - The problem is maintenance, when a change is done in the ltr style, it is very easy to adjust the rtl version when they are complementary. but, again, here it just won't do.
Comment #87
yoroy CreditAttribution: yoroy commentedBumpity
Comment #88
yoroy CreditAttribution: yoroy commented#82: toolbar+shortcut-rtl12.patch queued for re-testing.
Comment #89
casey CreditAttribution: casey commentedTested in all browsers available for windows.
Lookin' good!
Comment #90
casey CreditAttribution: casey commentedExcept for the small toolbar buttons...
Added display:block to toolbar-rtl.css:
Comment #91
aspilicious CreditAttribution: aspilicious commentedIe 7-8 looks bad. I will make a screenshot tomorow, also overlay is totally broken in IE6 with this. Don't know if it is related.
Bad review but I don't have a lot of time today.
Comment #92
yoroy CreditAttribution: yoroy commentedI believe you on your word though: needs work.
Comment #93
casey CreditAttribution: casey commentedI will need a screenshot though; not sure what is going wrong.
I found however that the user links aren't RTLed; Needs work anyway.
Comment #94
aspilicious CreditAttribution: aspilicious commentedWhoops I was wrong, I was looking at an overlay page, and overlay rtl is not in yet... (we need those rtl patches get in soon, very soon)
But it still needs work for the user links as you pointed out...
Comment #95
tsi CreditAttribution: tsi commentedOK, this is slightly better,
right to left order in the user links seems impossible in IE so using display: inline in
That makes it RTL at least for FF and doesn't break other browsers, needs testing.
I had to remove the line added in #90 and solved the resulting problem with some padding.
Comment #96
casey CreditAttribution: casey commentedAlmost there: shortcut links in rtl have same small buttons as described in #90 (at least FF).
Comment #97
Jeff Burnz CreditAttribution: Jeff Burnz commentedWhy use the longhand here when we use the shorthand in RTL? Also, the actual border-radius property should come after the browser extensions.
Powered by Dreditor.
Comment #98
tsi CreditAttribution: tsi commentedThis one uses the shorthand form in shortcut.css (#97)
@casey - I don't understand, see my screenshot taken from FF/linux
Comment #99
tsi CreditAttribution: tsi commentedComment #100
aspilicious CreditAttribution: aspilicious commentedtsi the black background has to be bigger on the shortcut bar.
Comment #101
bleen CreditAttribution: bleen commentedIn addition to aspilicious' comment in #100 ... IE7 (and IE6 has the same problems it seems) needs some love:
Comment #102
yoroy CreditAttribution: yoroy commented#98: toolbar+shortcut-rtl14.patch queued for re-testing.
Comment #103
reglogge CreditAttribution: reglogge commentedI see different issues with IE6/IE7 and also IE8.
IE6/IE7:
- Highlighting in the Toolbar looks fine
- Highlighting in the shortcuts is off by some pixels
IE8:
- No highlighting at all
Screenshot of IE7 attached. IE6 looks identical.
Comment #104
reglogge CreditAttribution: reglogge commentedJust a quickie: for the off-center highlighting in the shortcuts in IE7, you need to do this:
Without
display: block
the link doesn’t get its padding applied in IE7Comment #105
Jeff Burnz CreditAttribution: Jeff Burnz commentedI re-rolled the patch in #99 with a couple of very minor adjustments - mainly adding zoom:1; as this seems to trigger a display: block like behavior in IE6/7 and actually display: inline-block for everyone else. Removed some of the padding on links etc and I think we're either there, or one step closer.
This is not perfect but its very good - we can strive for perfection or live with some warts (not many). tsi has really done a great job here and I really take my hat off, its really something that you got this working so well in IE6/7 at all.
EDIT: BTW - the reason why hover styles do not work in IE8 is because of the filters to apply the shadow, if you take them out the hover styles come back for IE8 - so if we want them back we have to rethink the shadow for IE (in other words use an image).
Comment #106
reglogge CreditAttribution: reglogge commentedThis looks very good indeed. I tested this in all IE versions, FF, Chrome and Opera both on Mac and Windows.
Re the hovering styles in IE8:
Jeff is right, this is a problem with IE8 and IE8 only. We have several options here:
- Ignore it
- Check whether IE9 works (according to Microsoft http://msdn.microsoft.com/en-us/library/cc351024(VS.85).aspx , IE9 will support
box-shadow
) and if it does, ignore IE8 some more.- Implement something hackish since IE8 renders the backgrounds for hovered links in the toolbar when put into compatibility mode.
I tried playing around with the filter declarations in /modules/toolbar/toolbar.css
but to no avail. These were added in a monster issue #535066: Use CSS3 / IE filter to render toolbar shadow which I think nobody wants to revisit.
Overall, I think this is RTBC however. We can always deal with the IE8 issue, which is NOT germane to this particular issue addressing RTL, in a separate issue.
Great job Jeff and tsi!
Comment #107
webchickHm. My confidence level that we go back and fix this for IE 8 anytime soon if we don't fix it here is pretty low. And apparently hasn't been tested in IE 9. Sounds like this still needs some work.
Comment #108
reglogge CreditAttribution: reglogge commented@webchick: Please reconsider.
a) Is testing in IE9 a prerequisite for getting something committed? In the entire issue queue I find exactly 10 issues that even mention IE9.
On the Microsoft page which announces that IE9 will support box-shadow, they explicitly state this:
"Note All columns marked "Internet Explorer 9 Beta" are preliminary content. They may be changed substantially prior to commercial release of the software described herein."
In effect we can't know NOW what IE9 will or will not support once it gets released. And even if it will still support box-shadow we don't know how it will render the backgrounds on :hover - whether in some sort of Microsofty compatibility mode or whatever.
Anyway, I don't have Windows 7 or Vista, so I can't test IE9.
b) The rendering issue in IE8 is not limited to this here RTL issue. It's also present when viewing a site with a LTR language. I reopened #535066: Use CSS3 / IE filter to render toolbar shadow, which introduced this bug in the first place, since there are the people who should know much better about IE's arcane handling of -ms-filter, filter et.al. Maybe they can come up with a solution. I can't.
c) Right now there are quite a few RTL issues hanging in the issue queues, some of them waiting for each other to be committed, their patches breaking every time one of them finally gets committed, having to be rerolled over and over again. Committing this here would take one big chunk out of this cluster and allow other important issues to proceed more easily.
So please, let's just get this rightly major one in and be done with it.
Comment #109
Jeff Burnz CreditAttribution: Jeff Burnz commentedI'll gather some screenshots and test in IE9, lets take an objective look at the differences and see what we can do better.
Comment #110
JacineEDIT: Removing comment that was posted to the wrong issue.
Comment #111
JacineArgh... Wrong issue. Sorry.
Comment #112
Jeff Burnz CreditAttribution: Jeff Burnz commentedOK, I've tested in IE6, 7, 8, 9
Preliminary findings are that:
IE9 active tab styles don't work (not sure why as yet, works OK in LTR)
IE8 hover styles don't work (because of the filters, both LTR and RTL don't work)
IE7 and 6 - links have more horizontal space, we can fix this with some neg margin.
IE6 shadows bottom is not so good, the sprite image overflows - looks as good in RTL as it does in LTR.
Upshot? Would like to see the active styles working in IE9 and the hover style work in IE8 - at the moment I am not sure if the IE9 issues is releated to this or not (as in the IE8 hover issue is not, that's a filters issue).
The screenshot is from IE9 to 6 going down. Doesnt this highlight how IE6's time has come - why are we supporting 4 version of one browser?
Comment #113
aspilicious CreditAttribution: aspilicious commentedIf you find IE9 issues I recommend placing feedback on the feedback program. They changed their fieldset label rendering after a report of me. (just mention: drupal and millions of ugly sites cause of IE9 and don't forget to tell them how awesome IE9 is compared to IE8 and below)
Comment #114
Jeff Burnz CreditAttribution: Jeff Burnz commented@aspilicious - great idea, I've seen a few weird things, like Bartiks search button borked in RTL etc, I'll look that up.
One thing I think is very noticeable in those screenshots is how much better IE9's text rendering is, its just streets ahead -Lucida Grande looks like a dogs breakfast pre IE9, in fact I would be in favour of switching the font stack to verdana first for these versions, its really poor legibility as it is now.
Comment #115
aspilicious CreditAttribution: aspilicious commentedI saw the search button issue to...
Comment #116
ahmedwali CreditAttribution: ahmedwali commented#105: toolbar-shortcuts-rtl_740182.patch queued for re-testing.
Comment #117
aspilicious CreditAttribution: aspilicious commented@webchick @dries @anybody else
PLEASE commit at least 1! RTL patch.... (and start with this one)
Why?
1) It doesn't affect other stuff in drupal (it only adds css)
2) Few post above
3) I actually tested it today and made a special screenshot, don't let this effort be wasted
4) I'm willing to test/review followup patches if they are needed to improve things.
5) please?
I just rerolled the patch so this is RTBC if you want to fix the hover states in IE8 we probably have to wait till drupal 999.
Are we willing to wait that long for we actually fix drupal 7 for all those RTL users??
Comment #118
aspilicious CreditAttribution: aspilicious commentedBrowser order for the screenshot:
IE7
IE8
IE9 (rtl not fully supported yet in IE9 beta)
chrome
opera
firefox
Comment #119
aspilicious CreditAttribution: aspilicious commentedStatus...
Comment #120
tsi CreditAttribution: tsi commentedHover states issue in IE8 is not RTL specific (#535066: Use CSS3 / IE filter to render toolbar shadow).
So RTBC here too.
Comment #121
Dries CreditAttribution: Dries commentedReviewed and committed to CVS HEAD.
Comment #122
aspilicious CreditAttribution: aspilicious commentedDries == my hero
Comment #123
aspilicious CreditAttribution: aspilicious commentedWe lost track of the image that needed to get included.
Can you please replace the shortcut.png in the shortcut module?
Thnx!
Comment #124
webchickCommitted to HEAD. Thanks!