Skip to content

Commit

Permalink
Fixed issue 8137: Relevance equation cannot be edited for unknown rea…
Browse files Browse the repository at this point in the history
…sons
  • Loading branch information
c-schmitz committed Sep 23, 2013
1 parent 3ebae0a commit de373cd
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
9 changes: 4 additions & 5 deletions application/views/admin/survey/Question/editQuestion_view.php
Original file line number Diff line number Diff line change
Expand Up @@ -227,11 +227,10 @@
</li>
<li>
<label for='relevance'><?php $clang->eT("Relevance equation:"); ?></label>
<textarea cols='50' rows='1' id='relevance' name='relevance'
<?php if($eqrow['conditions_number']){ ?>
readonly='readonly' title='<?php $clang->eT("This question has one or more conditions. Delete them before edit the equation relevance."); ?>'
<?php } ?>
><?php echo $eqrow['relevance']; ?></textarea>
<textarea cols='50' rows='1' id='relevance' name='relevance' <?php if ($eqrow['conditions_number']) {?> readonly='readonly'<?php } ?>><?php echo $eqrow['relevance']; ?></textarea>
<?php if ($eqrow['conditions_number']) {?>
<span class='annotation'> <?php $clang->eT("Note: You can't edit the relevance equation because there are currently conditions set for this question."); ?></span>
<?php } ?>
</li>

<li id='Validation'>
Expand Down
1 change: 1 addition & 0 deletions styles/blobblueish/adminstyle.css
Original file line number Diff line number Diff line change
Expand Up @@ -385,6 +385,7 @@ input[disabled], textarea[disabled], select[disabled], button[disabled] {
.annotation {
font-size: 10px;
color: #E00;
vertical-align: top;
}

.errortitle {
Expand Down
1 change: 1 addition & 0 deletions styles/gringegreen/adminstyle.css
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,7 @@ textarea {
.annotation {
font-size: 10px;
color: #FF9D00;
vertical-align: top;
}

.errortitle {
Expand Down

3 comments on commit de373cd

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just : i really prefer "title" system than a new span ;)

@tpartner
Copy link
Collaborator

@tpartner tpartner commented on de373cd Sep 24, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One time is better

All are updatable by admin template :)

Please sign in to comment.