#3903 closed defect (bug) (wontfix)
js for showing #whats-new-options on textarea focus has wrong value
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | minor | Version: | 1.5.2 |
Component: | Templates | Keywords: | close |
Cc: |
Description
In this bit of global.js:
/* Textarea focus */
jq('#whats-new').focus( function(){
jq("#whats-new-options").animate({height:'40px'});
jq("form#whats-new-form textarea").animate({height:'50px'});
jq("#aw-whats-new-submit").prop("disabled", false);
});
The .animate({height:'40px'}); value needs to be more like 60px. I'm on a Mac, and if the submit button and the "Post in" drop-down both need to show, 40px is too short. See attached image.
Attachments (1)
Change History (7)
#2
in reply to:
↑ 1
@
13 years ago
- Severity changed from normal to minor
Replying to r-a-y:
The JS is applicable to the bp-default theme.
Are you using a custom theme? Or are you using bp-default on a mobile device by chance?
I'm using a TwentyEleven child theme install. I haven't had to modify the bp-default theming—it works really well out-of-the-box. Using Firebug, I can verify that the css for all the elements, including text size, in this area is coming from bp-default.
This is on a desktop browser.
BTW, I'm amazed at how well WP/BP+child theming work together.
#3
@
13 years ago
- Keywords close added
Since this issue is applicable to your TwentyEleven child theme, I'd recommend rolling your own JS and using unbind().
Then you could set your own dimensions for the "What's New" auto-expander.
#4
follow-up:
↓ 5
@
13 years ago
- Component changed from Core to Theme
- Milestone Awaiting Review deleted
- Resolution set to wontfix
- Status changed from new to closed
At the end of the day, BP-Default's javascript is built for BP-Default. You're going to have to customise this in your theme; r-a-y suggests one approach. We're working towards not having so much useful stuff in the core theme, i.e. we might do something like a common javascript file.
#5
in reply to:
↑ 4
@
13 years ago
Replying to DJPaul:
At the end of the day, BP-Default's javascript is built for BP-Default. You're going to have to customise this in your theme; r-a-y suggests one approach. We're working towards not having so much useful stuff in the core theme, i.e. we might do something like a common javascript file.
I think your future plan is a good one; splitting some of the pieces out so BP isn't so monolithic would make it easier to customize.
Thanks!
The JS is applicable to the bp-default theme.
Are you using a custom theme? Or are you using bp-default on a mobile device by chance?