[#MDL-9535] Only half of Send Message button displays in Send Message. - Moodle Tracker

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Won't Fix
Affects Version/s: 1.8
Fix Version/s: None
Component/s: Messages
Labels:
None
Environment:
Firefox 2.0.0.3

Affected Branches:
MOODLE_18_STABLE

Description

Only half of the Send Message button displays in the "send message" pop-up window. This is a Firefox issue not IE.

Issue Links

This issue duplicates:
MDL-14280	No "Send message" button.

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Carlos Vargas added a comment - 27/Apr/07 4:06 AM

I found a fix ...

In moodle/message/send.php, I deleted <br /> on LINE 105, and that took care of the problem:
from...
echo '<br /><input type="submit" value="'.get_string('sendmessage', 'message').'" />';
to...
echo '<input type="submit" value="'.get_string('sendmessage', 'message').'" />';

Show

Carlos Vargas added a comment - 27/Apr/07 4:06 AM I found a fix ... In moodle/message/send.php, I deleted <br /> on LINE 105, and that took care of the problem: from... echo '<br /><input type="submit" value="'.get_string('sendmessage', 'message').'" />'; to... echo '<input type="submit" value="'.get_string('sendmessage', 'message').'" />';

Hide

Permalink

Jeffrey Silverman added a comment - 20/Jun/08 9:59 PM

I have had clients describing this issue in Firefox and in MSIE 7. For some, the "Send" button is pushed so far down the screen as to be not visible at all.

To fix it, I increased the size of the messaging window and the relative size of the frames within the messaging window. (I must note that I actually wasn't able to test whether this worked because I was not exhibiting the issue)

The changes I made touched the following files:

message/lib.php
user/view.php

(There might be more places where a popup link to a send message window exists)

Anyway, I'd like to make another recommendation that the size of the popup windows for messaging not be hard-coded into four or five different places but instead be define()ed or set to a variable somewhere in a library or at the top of the files in which the popup js link occurs.

Thank you!

Show

Jeffrey Silverman added a comment - 20/Jun/08 9:59 PM I have had clients describing this issue in Firefox and in MSIE 7. For some, the "Send" button is pushed so far down the screen as to be not visible at all. To fix it, I increased the size of the messaging window and the relative size of the frames within the messaging window. (I must note that I actually wasn't able to test whether this worked because I was not exhibiting the issue) The changes I made touched the following files: message/lib.php user/view.php (There might be more places where a popup link to a send message window exists) Anyway, I'd like to make another recommendation that the size of the popup windows for messaging not be hard-coded into four or five different places but instead be define()ed or set to a variable somewhere in a library or at the top of the files in which the popup js link occurs. Thank you!

Hide

Permalink

Michael de Raadt added a comment - 05/Jan/12 1:45 PM

Thanks for reporting this issue.

We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

Michael d;

lqjjLKA0p6

Show

Michael de Raadt added a comment - 05/Jan/12 1:45 PM Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; lqjjLKA0p6

Hide

Permalink

Michael de Raadt added a comment - 06/Feb/12 4:25 PM

I'm closing this issue as it has become inactive and does not appear to affect a current supported version. If you are encountering this problem or one similar, please launch a new issue.

Show

Michael de Raadt added a comment - 06/Feb/12 4:25 PM I'm closing this issue as it has become inactive and does not appear to affect a current supported version. If you are encountering this problem or one similar, please launch a new issue.

People

Assignee:

Mathieu Petit-Clair

Reporter:

Carlos Vargas

Participants:

Carlos Vargas, Jeffrey Silverman, Mathieu Petit-Clair, Michael de Raadt

Vote (1)

Watch (2)

Dates

Created:

24/Apr/07 1:35 PM

Updated:

06/Feb/12 4:25 PM

Resolved:

06/Feb/12 4:25 PM