Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the design and the insolation of the tab bar and add the Add option #7

Merged
merged 3 commits into from
Jan 11, 2020

Conversation

david-minaya
Copy link
Owner

Change the method of insolation

The method used to insolate the tab bar was changed. Now we use iframe instead of the "all" property. For more details about this decision see the issue #6.

Improve the design of the tab bar

The design of the tab bar was improved:

Old
old tab bar

New
new tab bar

See the commit 31a239d for more details.

Add option

Add the Add option. This option lets the user add new tabs to the tab bar.

QA

  1. Compile the extension npm run build
  2. Open the tab bar page. To open this page, open the popup of the tab bar and click in the button Open page.
  3. Open a tab group
  4. In the new page opened do click in the Add option of the tab bar (+). If a new tab is added to the tab bar then all work well.

Now the design of the tab bar is a float tab bar of white color,
instead of a tab bar gray anchored at the bottom of the page. Also,
now, when a tab is selected in the tab bar, this tab shows a border
of the color blue in its bottom, instead of changing its background
color to the red color.
The method used to isolate the tab bar using the "all" property was
changed to use an iframe. For more details on this decision, see the
issue #6.
When the user clicks in the Add option, a new tab is added to the
tab bar.

Fixes:

- Now only a tab can be selected at a time, before, all tabs that
  were added or clicked were selected. Now only the new tab or the
  clicked tab is selected.
@david-minaya david-minaya self-assigned this Jan 11, 2020
@david-minaya david-minaya merged commit ac58f5c into develop Jan 11, 2020
@david-minaya david-minaya deleted the tab-bar branch January 11, 2020 02:14
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.

1 participant