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

feat(scaffold-navigation): upgrade to bootstrap v4 and fontawesome v5 #1167

Merged
merged 5 commits into from
Mar 11, 2020

Conversation

3cp
Copy link
Member

@3cp 3cp commented Mar 10, 2020

This has been a long requested upgrade.

@3cp
Copy link
Member Author

3cp commented Mar 10, 2020

@shahabganji could you help to have a look of test_docker_2? I cannot reproduce that protractor error locally.

1) aurelia skeleton app should navigate to users page
  Message:
    Failed: element not interactable

@shahabganji
Copy link
Contributor

@3cp Sure I will

@3cp 3cp force-pushed the bs4-scaffold-navigation branch 3 times, most recently from 9beb9d6 to fbc6aeb Compare March 11, 2020 02:58
@@ -56,7 +56,8 @@ const config = {
'--disable-renderer-backgrounding',
'--disable-device-discovery-notifications',
'--no-sandbox',
'--disable-dev-shm-usage'
'--disable-dev-shm-usage',
'--window-size=1024,768'
Copy link
Member Author

Choose a reason for hiding this comment

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

@shahabganji this fix did the job. The reason for element not interactable is that the protractor browser window size is too small.

Nobody knows why the default screen size is so small inside docker(or maybe vm). Explicit window size does the trick.

Copy link
Member Author

Choose a reason for hiding this comment

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

When element is off screen, protractor says it's not interactable.

@3cp 3cp merged commit 4652e0e into master Mar 11, 2020
@3cp 3cp deleted the bs4-scaffold-navigation branch March 11, 2020 03:49
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.

2 participants