Issue 81076 - Round the number of rows result set displayed for percentage filters.
Summary: Round the number of rows result set displayed for percentage filters.
Alias: None
Product: Calc
Classification: Application
Component: code (show other issues)
Version: OOo 2.1
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
Depends on:
Blocks: 72764
  Show dependency tree
Reported: 2007-08-28 09:20 UTC by gaozm
Modified: 2017-05-20 11:13 UTC (History)
5 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

81076.path (689 bytes, text/plain)
2007-08-28 10:13 UTC, gaozm
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description gaozm 2007-08-28 09:20:19 UTC
1. Create a new Calc
2. Input characters into any column,for example between a1 and a21 input       
figures from 1 to 21
3.Select the whole column a1,data->filter->Auto Filter->Yes, Click the black 
arrow,select the 'Standard',condition->smallest %; value->10; ok
Result: It will show 3 rows(except the column header).
Somebody hopes it should show 2 rows(except the column header).
Comment 1 liujiaxiang 2007-08-28 09:45:06 UTC
change the milestone
Comment 2 gaozm 2007-08-28 10:13:21 UTC
Created attachment 47819 [details]
Comment 3 ooo 2007-08-28 12:13:49 UTC
Sorry, the patch doesn't take the correct approach. It eliminates the row header
detection so a row header would be considered to be part of the data, instead of
rounding the result value.

And please submit patches against the original upstream source, not some
RedOffice modified source. The attached patch won't apply on upstream source.

Comment 4 frank 2007-08-28 12:50:34 UTC
target later as no correct patch is attached.
assigned to er to keep an eye on it for integration if a correct patch is supplied.
Comment 5 ooo 2007-08-28 13:07:21 UTC
Seems setting the target was omitted.
Comment 6 pagalmes.lists 2008-02-20 09:42:48 UTC
shoulnd't that be set as a PATCH issue type?
Comment 7 ooo 2008-02-21 00:40:32 UTC
@pagalmes: no, as the patch isn't correct, see my comment in #desc4 from Tue Aug
28 11:13:49 +0000 2007.
Comment 8 Marcus 2017-05-20 11:13:51 UTC
Reset assigne to the default "".