PortScanner icon indicating copy to clipboard operation
PortScanner copied to clipboard

[BUG] [Frontend] Adjust the Timer Size

Open ishika1727 opened this issue 3 years ago • 24 comments

The size of the timer/clock should vary according to the screen size. In mobile view, the timer should appear below the input fields and the button.

ishika1727 avatar Aug 16 '20 18:08 ishika1727

@ishika1727 can I work on this issue.

Amulya-coder avatar Aug 29 '20 18:08 Amulya-coder

@Amulya-coder please go ahead 👍 Create a PR by wednesday EOD. Also, if you are participant of Hakincodes please introduce yourself in the Welcome to Hakincodes thread. 👋 It will help us keep track of your progress and contributions during the period. 😄

ishika1727 avatar Aug 29 '20 19:08 ishika1727

BUG2

Amulya-coder avatar Aug 30 '20 09:08 Amulya-coder

@ishika1727 Can you tell me a little more about what timer you should indicate the red or blue and what you mean by adjust the Timer size its look fine in 217px Can you through some light on this so that I can understand.

Amulya-coder avatar Aug 30 '20 09:08 Amulya-coder

@Amulya-coder you have to reduce the size of the time shown on the right hand side(time taken to fetch the results). Refer the README.md for steps to run the application and reduce the timer size i.e. the seconds and minutes shown

image

ishika1727 avatar Aug 30 '20 11:08 ishika1727

@Amulya-coder you have to reduce the size of the time shown on the right hand side(time taken to fetch the results). Refer the README.md for steps to run the application and reduce the timer size i.e. the seconds and minutes shown

image

@ishika1727 thank you for your quick response I got it . I read the README.md and follows each and every steps and install all the dependencies but there is another problem came when I run the python src/scanner.py command in my terminal it will show some error I think something have not done correctly I will also attach the screenshot of this below. Can you please help me out for fixing this.

Amulya-coder avatar Aug 30 '20 15:08 Amulya-coder

BUG3

Amulya-coder avatar Aug 30 '20 15:08 Amulya-coder

@ishika1727 can you help me out for fixing this error that is shown in the terminal .

Amulya-coder avatar Aug 31 '20 05:08 Amulya-coder

@Amulya-coder try running with python2 in master branch

image

ishika1727 avatar Aug 31 '20 08:08 ishika1727

@ishika1727 the Port Scanner runs and it will shows all the open ports in the terminal but the website was not shown in a correct manner when I open in the browser it will displayed like this why?

Amulya-coder avatar Aug 31 '20 15:08 Amulya-coder

Bug4

Amulya-coder avatar Aug 31 '20 15:08 Amulya-coder

@Amulya-coder i ll verify and get back to you.

ishika1727 avatar Aug 31 '20 18:08 ishika1727

@Amulya-coder I can see you are directly walking into src/templates/index.html. Follow these steps here to run the program locally and enter the url which the program returns (step 6).

Kashish121 avatar Sep 01 '20 11:09 Kashish121

@Amulya-coder please refer to feature/frontend branch rather than the master branch and make changes in that only.

ishika1727 avatar Sep 01 '20 12:09 ishika1727

@Kashish121 I follow these steps as you mentioned it will running and returned the URL but when I click on the URL it opens the website in this way

PIC1

Amulya-coder avatar Sep 01 '20 17:09 Amulya-coder

It's not showing in the correct way as I want

Amulya-coder avatar Sep 01 '20 17:09 Amulya-coder

@Amulya-coder give the following inputs and then start scanning with the scan now button. The timer will be displayed.

Host to scan: localhost low: 1 high: 20

ishika1727 avatar Sep 01 '20 17:09 ishika1727

@ishika1727 still not shows the timer it will show range and below it time

PIC3

Amulya-coder avatar Sep 01 '20 18:09 Amulya-coder

@ishika1727 why it is not showing in the way like that it shows in your's.

PIC2

Amulya-coder avatar Sep 01 '20 18:09 Amulya-coder

@Amulya-coder please don't run src/templates/index.html directly. Follow these steps here to run the program locally and then make the changes in frontend files. The backend and frontend are connected using flask in the mainscanner.py that is why directly running it it giving different result.

ishika1727 avatar Sep 01 '20 18:09 ishika1727

@Amulya-coder that's because this screen shot is out dated. We have merged new changes few hours ago, I suggest you clone the repository again and try running. Also, the timer will appear only during the scanning process. Not before and after.

@ishika1727 why it is not showing in the way like that it shows in your's.

PIC2

Kashish121 avatar Sep 01 '20 18:09 Kashish121

@ishika1727 In timer size I have seen directly that it's size is bigger because of not proper font sizing so to adjust it I have to particularly find that file and then make changes in it so what's your suggestions.

Amulya-coder avatar Sep 02 '20 07:09 Amulya-coder

@vinitshahdeo @Kashish121 I have made the PR .Can you please verify and merge it.

Amulya-coder avatar Oct 02 '20 09:10 Amulya-coder

@vinitshahdeo I have made the required changes please review it . Thanks.

Timer Adjusted #100

Amulya-coder avatar Oct 03 '20 03:10 Amulya-coder