SKIP: revert tabulate-seqs changes but still allow FeatureData[LinkedSequence] as input (#367)

<!-- PR guidance and examples:
https://github.com/rachis-org/governance/blob/main/pr_template_guidelines.md
-->
<!-- rachis Generative AI Policy:
https://github.com/rachis-org/governance/blob/main/ai_policy.md -->

# Description
<!-- REQUIRED: Briefly describe this PR, or link the issue it closes.
-->
Reverts changes from #366 

# AI Disclosure
<!-- REQUIRED: Check exactly one option. -->

- [x] NO AI USED.
- [ ] AI USED.

Author: Macabe222

q2_feature_table/_summarize/_visualizer.py

@@ -42,20 +42,12 @@ def tabulate_seqs(output_dir: str, data: DNAIterator,
         for sequence in data:
             skbio.io.write(sequence, format='fasta', into=fh)
             str_seq = str(sequence)
-            seqs = str_seq.split(' ')
-            urls = [_blast_url_template % seq for seq in seqs]
-            seq_id = sequence.metadata['id']
-            display_sequences.add(seq_id)
-
-            sequences[seq_id] = {
-                'len': len(str_seq)
-            }
-            if len(seqs) == 2:
-                sequences[seq_id]['url_and_seq'] = zip(
-                    urls, seqs, ['forward', 'reverse']
-                )
-            else:
-                sequences[seq_id]['url_and_seq'] = zip(urls, seqs)
+            seq_len = len(str_seq)
+            display_sequences.add(sequence.metadata['id'])
+            sequences[sequence.metadata['id']]\
+                = {'len': seq_len,
+                   'url': _blast_url_template % str_seq,
+                   'seq': str_seq}
 
     if metadata is not None:
         metadata_df = metadata.to_dataframe()

q2_feature_table/_summarize/tabulate_seqs_assets/index.html

@@ -133,25 +133,7 @@ <h3>Sequence Table</h3>
             {% endif %}
             {% if sequence in data %}
             <td>{{ data[sequence].len }}</td>
-            <td><samp>
-                {% for item in data[sequence].url_and_seq %}
-                  {% if item|length == 3 %}
-                    {% set link, seq, direction = item %}
-                    {% if direction == 'reverse' %}
-                        <br>
-                    {% endif %}
-                    {{ direction }}:
-                    <a target="_blank" href="{{ link }}" rel="noopener noreferrer">
-                      {{ seq }}
-                    </a>
-                  {% else %}
-                    {% set link, seq = item %}
-                    <a target="_blank" href="{{ link }}" rel="noopener noreferrer">
-                      {{ seq }}
-                    </a>
-                  {% endif %}
-                {% endfor %}
-            </samp></td>
+            <td><samp><a target="_blank" href="{{ data[sequence].url }}" rel="noopener noreferrer">{{ data[sequence].seq }}</a></samp></td>
             {% else %}
             <td>-</td>
             <td>-</td>

q2_feature_table/tests/test_summarize.py

@@ -50,7 +50,7 @@ def test_basic(self):
             self.assertTrue(os.path.exists(expected_fp))
             with open(expected_fp) as fh:
                 file_text = fh.read()
-                self.assertTrue('ACGT' in file_text)
+                self.assertTrue('ACGT</a>' in file_text)
                 self.assertTrue('<td>4</td>' in file_text)
                 self.assertTrue('<td>seq2</td>' in file_text)