Skip to content

Distinguish product and test issues bugref#708

Merged
coolo merged 1 commit into
os-autoinst:masterfrom
okurz:feature/poo_icon
Jun 6, 2016
Merged

Distinguish product and test issues bugref#708
coolo merged 1 commit into
os-autoinst:masterfrom
okurz:feature/poo_icon

Conversation

@okurz
Copy link
Copy Markdown
Member

@okurz okurz commented Jun 6, 2016

Progress is used to track test issues, bugzilla for product issues, at least
for SUSE/openSUSE. This might be a lot different for other groups so a more
flexible approach could be thought of but probably it is not as easy as
just distinguishing on the issue tracker URL being used, e.g. if a group
uses just one issue tracker but different components or issue types to
differentiate, it is not that easy. That is why this simple solution is
done here, first. At least others can follow the approach and have a test
and CSS rule available ;-)

Example screenshot:
openqa_bolt_icon_for_progress_issue

Alternative 1: Inverted bug icon, vote for this, if you like it more than what has been currently implemented:
openqa_inverted_red_bug_for_progress_issue

<span id="bug-<%= $jobid %>">
<a href="<%= bugurl_for $bug %>">
<i class="test-label label_bug fa fa-bug" title="Bug(s) referenced: <%= $bug %>"></i>
<i class="test-label label_bug fa <%= $bug =~ /poo#/ ? 'fa-bolt' : 'fa-bug' %>" title="Bug(s) referenced: <%= $bug %>"></i>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to move that code into a helper

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do, code update is not related. Need to rebase anyway, I guess

@okurz okurz force-pushed the feature/poo_icon branch from 0aa9208 to 869ec10 Compare June 6, 2016 09:42
Progress is used to track test issues, bugzilla for product issues, at least
for SUSE/openSUSE. This might be a lot different for other groups so a more
flexible approach could be thought of but probably it is not as easy as
just distinguishing on the issue tracker URL being used, e.g. if a group
uses just one issue tracker but different components or issue types to
differentiate, it is not that easy. That is why this simple solution is
done here, first. At least others can follow the approach and have a test
and CSS rule available ;-)
@okurz okurz force-pushed the feature/poo_icon branch from 869ec10 to a80a5bb Compare June 6, 2016 09:56
@okurz
Copy link
Copy Markdown
Member Author

okurz commented Jun 6, 2016

updated:

  • rebased
  • add helper bugicon_for

@sysrich
Copy link
Copy Markdown
Member

sysrich commented Jun 6, 2016

I prefer the lightning bolt than the upside down bug

@okurz
Copy link
Copy Markdown
Member Author

okurz commented Jun 6, 2016

good, then nothing needs to be changed :-)

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 65.959% when pulling a80a5bb on okurz:feature/poo_icon into d758ad0 on os-autoinst:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 65.936% when pulling 869ec10 on okurz:feature/poo_icon into d758ad0 on os-autoinst:master.

@coolo coolo merged commit 510d499 into os-autoinst:master Jun 6, 2016
@okurz okurz deleted the feature/poo_icon branch November 13, 2018 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants