Project

General

Profile

Actions

Defect #2414

open

'Add News Button' in 'Add news'

Added by Karl Heinz Marbaise almost 16 years ago. Updated over 3 years ago.

Status:
Reopened
Priority:
Normal
Assignee:
-
Category:
News
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Affected version:

Description

I have observed a little 'Bug', cause if i going to News and click on 'Add news' I'm forwarded to the 'Add news' area but the 'Add news' Button is displayed anymore.
My opinion is not to display the 'Add news' Button there. Take a look at the attached screen shot.

May be this is a feauture? or already fixed on a newer revision ?

  • Revision 2081 on trunk
    my environment:
    minden205:/usr/local/vhosts/redmine # ruby script/about
    About your application's environment
    Ruby version              1.8.6 (x86_64-linux)
    RubyGems version          1.3.0
    Rails version             2.1.1
    Active Record version     2.1.1
    Action Pack version       2.1.1
    Active Resource version   2.1.1
    Action Mailer version     2.1.1
    Active Support version    2.1.1
    Application root          /usr/local/vhosts/redmine
    Environment               development
    Database adapter          mysql
    Database schema version   101
    

Files

redmine-news-add.jpg (32.8 KB) redmine-news-add.jpg Add news Item Screenshot. Karl Heinz Marbaise, 2008-12-29 18:40
fix_add_news_button.patch (790 Bytes) fix_add_news_button.patch Alexander Meindl, 2021-05-07 11:04

Related issues

Related to Redmine - Defect #12824: Useless "edit" link in workflow menuClosedJean-Philippe Lang

Actions
Actions #1

Updated by Karl Heinz Marbaise almost 16 years ago

  • Start date deleted (2008-12-29)
Actions #2

Updated by Karl Heinz Marbaise almost 16 years ago

Hi,
rechecked revision:

$/usr/local/vhosts/redmine # svn info
Path: .
URL: http://redmine.rubyforge.org/svn/branches/0.8-stable
Repository Root: http://redmine.rubyforge.org/svn
Repository UUID: e93f8b46-1217-0410-a6f0-8f06a7374b81
Revision: 2201
Node Kind: directory
Schedule: normal
Last Changed Author: jplang
Last Changed Rev: 2200
Last Changed Date: 2008-12-29 16:50:08 +0100 (Mon, 29 Dec 2008)

The same behaviour.

Actions #3

Updated by Etienne Massip over 13 years ago

  • Target version set to Candidate for next minor release
Actions #4

Updated by Eugene Sypachev over 13 years ago

  • Status changed from New to Resolved

This will fix the problem:

Index: index.rhtml
===================================================================
--- index.rhtml    (revision 6120)
+++ index.rhtml    (working copy)
@@ -2,7 +2,7 @@
 <%= link_to(l(:label_news_new),
             new_project_news_path(@project),
             :class => 'icon icon-add',
-            :onclick => 'Element.show("add-news"); Form.Element.focus("news_title"); return false;') if @project && User.current.allowed_to?(:manage_news, @project) %>
+            :onclick => 'Element.show("add-news"); Element.hide("contextual"); Form.Element.focus("news_title"); return false;') if @project && User.current.allowed_to?(:manage_news, @project) %>
 </div>

 <div id="add-news" style="display:none;">
@@ -17,7 +17,7 @@
                      :update => 'preview',
                      :with => "Form.serialize('news-form')" 
                    }, :accesskey => accesskey(:preview) %> |
-<%= link_to l(:button_cancel), "#", :onclick => 'Element.hide("add-news")' %>
+<%= link_to l(:button_cancel), "#", :onclick => 'Element.hide("add-news"); Element.show("contextual");' %>
 <% end if @project %>
 <div id="preview" class="wiki"></div>
 </div>

Sorry, don't found the way to commit changes to SVN repo.

Actions #5

Updated by Etienne Massip over 13 years ago

  • Status changed from Resolved to New
Actions #6

Updated by Daniel Felix almost 12 years ago

Well, this seems to be fixed in the current version of redmine (trunk and version 2.2).

I think this could be closed, or does anyone else encounter this problem?

Actions #7

Updated by Daniel Felix almost 12 years ago

  • Status changed from New to Closed
  • Target version deleted (Candidate for next minor release)

I'm closing it, see my prior comment.

Actions #8

Updated by Mischa The Evil almost 12 years ago

  • Status changed from Closed to Reopened

This is still an issue IMO. It's about the fact that the "Add news" button is displayed while adding a new news item (thus being on the new news view). This also applies to two other new views: 'New message' and 'New document'.

This can be "reproduced" on *.m.redmine.org (and rm.o) as such. I think we can argue about whether it's a defect or not. The 'Add x'-functionality is not broken in any way, it's just the lone fact that the button is displayed on those views. I think this is more a design-consequence rather than a defect - not completely sure here, so correct me if I'm wrong.
More general: I think this is more of a FR to request improvements regarding the consistency of the UI, more specific the mentioned 'New x views'. What do you think?

Actions #9

Updated by Daniel Felix almost 12 years ago

Oh, yes you're right. The button was just hidden behind an elemente as I tested it on my testsystem.

This relates to #12824.

Actions #10

Updated by Alexander Meindl over 3 years ago

Hi,

I created a new patch to fix the problem. With my solution a user with manage news permissions (in any project) can see the "add news" button on global news index.

Without this patch, everyone can see "add news" button on global news index - even without permissions.

Actions #11

Updated by Go MAEDA over 3 years ago

  • Target version set to Candidate for next minor release

Alexander Meindl wrote:

I created a new patch to fix the problem. With my solution a user with manage news permissions (in any project) can see the "add news" button on global news index.

Without this patch, everyone can see "add news" button on global news index - even without permissions.

Thank you for detecting and reporting the issue.

The bug you is a new one in Redmine 4.2.0 (#33167) and is different from this one. I have opened #35308 for the bug and attached the patch.

Actions #12

Updated by Go MAEDA over 3 years ago

  • Target version deleted (Candidate for next minor release)
Actions

Also available in: Atom PDF