[#5924] - fixed admin mobile menu toggle not working
- Closed
- 31 Jan 2015
- Medium
- Build: staging
- # 5924
- Diff
- C-Lodder:mobile-menu
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
jissues-bot
- | Labels |
Added:
?
|
||
| Rel_Number | ⇒ | 5912 | |
| Relation Type | ⇒ | Related to |
@brianteeman - yeah, I ended up deleting and reforking when I got home. Does this cause any issues for the commit I made before?
I think you have deleted the wrong one. If you go to github it cannot fund
your code for this PR so it cannot be merged
It should be fine for merging (github shows a valid merge button anyhow) but if there are any issues with this pr you probably won't be able to update it and will have to make another new one.
I tested it originally before the LESS was added. As the LESS compiles (or
George couldnt have tested it correctly) I thik mine and all ther previous
tests should count and this can be merged
On 30 January 2015 at 21:51, George Wilson notifications@github.com wrote:
@test https://github.com/test replicated the initial menu issue and
patch fixes it. One more tester—
Reply to this email directly or view it on GitHub
#5924 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
| Category | ⇒ | Templates (admin) UI/UX |
| Status | Pending | ⇒ | Ready to Commit |
moving to RTC since we have two tests here Thanks to all here!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5924.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-31 00:21:34 |
Did you delete the wrong repo?