Rowspan not rendering the rows properly

Question

0.00/5 (No votes)

See more:

I have the following View in my Web Application. I am unable to use the rowspan properly. I can't find what property I am missing to get the row in the third column to split into the number based on the count.

My count has the exact number of properties. only two rows are rendered properly and the last one is not rendered. I have also tried using @: . But nothing worked.

View

HTML

@foreach (var item in Model.OptionValues)
    {            
      <tr>
        <td rowspan="@item.SetValue.Count" >@item.OptionVal</td>
        <td rowspan="@item.SetValue.Count">@item.OptionValueID</td>
        @{var set = item.SetValue.FirstOrDefault(x => x.OptionValueID == item.OptionValueID);}
        @if (set != null)
        {
            for (int i = 0; i < item.SetValue.Count; i++)
            {
               <tr>
                  <td>@set.TcSet.SetName</td>
                  <td> @set.Value</td>
                  <td>@set.Status</td>
               </tr>
            }

        }
      </tr>
    }

I am not getting all the rows rendered properly in a single column. the last row is shown along with the main table.

Posted 8-Oct-15 2:52am

vini vasundharan

Add a Solution

Comments

[no name] 8-Oct-15 11:00am

Firstly @item.SetValue.Count has correct value?

Secondly as per your logic whatever HTML code is generating, please deeply look into that. Make sure that HTML code is properly generated. If you have any doubt please provide what your HTML code generating with code.

vini vasundharan 9-Oct-15 1:38am

i have already checked if count has the right value. It has the right value. The code doesnt display the results as required. the logic in the HTML block is wrong but it is right to generate a table with rowspan

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Richard Deeming · Answer 1 · 2015-10-08T08:25:00

You're generating invalid HTML - you can't nest a <tr> inside another <tr>.

Try something like this instead:

HTML

@foreach (var item in Model.OptionValues)
{            
    <tr>
        <td rowspan="@item.SetValue.Count" >@item.OptionVal</td>
        <td rowspan="@item.SetValue.Count">@item.OptionValueID</td>
		
        @{var set = item.SetValue.FirstOrDefault(x => x.OptionValueID == item.OptionValueID);}
		
        @if (set != null)
        {
            <td>@set.TcSet.SetName</td>
            <td> @set.Value</td>
            <td>@set.Status</td>
			
            for (int i = 1; i < item.SetValue.Count; i++)
            {
                </tr>
                <tr>
                    <td>@set.TcSet.SetName</td>
                    <td> @set.Value</td>
                    <td>@set.Status</td>
            }
        }
    </tr>
}

vini vasundharan · Answer 2 · 2015-10-08T19:42:00

HTML

@foreach (var item in Model.OptionValues)
        {
            //var row = @item.SetValue.Count + 1;

            @:<table><tbody><tr>
                @:<td rowspan="@item.SetValue.Count">@item.OptionVal</td>
                @:<td rowspan="@item.SetValue.Count">@item.OptionValueID</td>
                var set = item.SetValue.FirstOrDefault(x => x.OptionValueID == item.OptionValueID);
                if (set != null)
                {

                    for (int i = 0; i < item.SetValue.Count; i++)
                    {
                        if (i == 0)
                            {
                                @:<td>@set.TcSet.SetName</td>
                                @:<td> @set.Value</td>
                                @:<td>@set.Status</td>
                                @:</tr></tbody></table>
                            }

                            else
                            {

                            
                                @:<tr>
                                @:<td>@set.TcSet.SetName</td>
                                @:<td> @set.Value</td>
                                @:<td>@set.Status</td>
                           
                        

                    }

                }
             @:</tr>
        }

I did it this way to get it right. Any other better approach to get it right?

Rowspan not rendering the rows properly

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0