Bug 454038
| Summary: | Additional columns available in query resultsgroups doesn't wrap | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Community] Bugzilla | Reporter: | Noura El hawary <nelhawar> | ||||
| Component: | User Interface | Assignee: | Noura El hawary <nelhawar> | ||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | |||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | 3.2 | ||||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2008-07-09 16:19:40 UTC | Type: | --- | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 406071 | ||||||
| Attachments: |
|
||||||
|
Description
Noura El hawary
2008-07-04 01:54:27 UTC
Created attachment 310972 [details] v1 wrapping addtional columns in buglist.cgi as an example of this problem please see, https://bz-web2-test.devel.redhat.com/buglist.cgi?bug_id=440169&columnlist=bug_severity%2Cpriority%2Crep_platform%2Cassigned_to%2Cbug_status%2Cresolution%2Cop_sys%2Cdependson%2Cblockedby%2Cshort_desc&field-1-0-0=bug_id&query_format=advanced&remaction=&type-1-0-0=anyexact&value-1-0-0=440169&query_based_on= attached is a patch to fix it and add the wrap option to those columns I am not sure if this is the right place to the the option or it should be added to an extension, i can not find the hook "list_table_abbrev" that is mentioned in template/en/default/list/table.html.tmpl ,, also the patch in the bug 406151 doesn't include this hook. Please review when you can. Thanks, Noura Comment on attachment 310972 [details]
v1 wrapping addtional columns in buglist.cgi
If we did it with an extension it would be
extensions/<extension_name>/template/en/list/table-list_table_abbrev.html.tmpl
but since we have the columns in the Bugzilla code in several places already
such as Bugzilla/Search.pm, colchange.cgi, field-descs.none.tmpl, etc. I think
we can leave it in the templates as well for now. So your patch looks fine to
me.
Just put REDHAT EXTENSION around the section similar to how we have it in
template/en/default/global/field-descs.none.tmpl. And then check it in.
Dave
Thanks for the review Dave ,, all committed to cvs now. Noura |